[Bug 225244] Merge Review: amtu

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Merge Review: amtu


https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=225244





------- Additional Comments From kevin@xxxxxxxxx  2007-02-04 10:20 EST -------
In reply to comment #3: 

>I don't know where IBM distributes it from. I work directly with them on CC
>evals and the tarball is emailed to me.

Could you perhaps ping them to update the sourceforge site with the latest?
Then it would be easy to check against upstream. 

>>Setup should have -q on it?
>
>I suppose it could. Doesn't hurt anything either way.

True, just more in line with all the other packages... 

>>Why aren't you using %configure?
>
>In my experience, some packages will not build with %configure. We might be
>able to change it.

Can you try and do so?

>>Why is this 750?
>
>Because it needs to be. It requires privileges to run. If it were available to
>accounts that could run it and fail, the audit requirements are higher.

Sounds reasonable. 

>>This should be fixed to not have doc line endings...
>
>Attach a patch please.

You can do it in the spec... just add: 

%{__sed} -i 's/\r//' AMTUHowTo.txt

>>3. Why the compiler setting stuff in build?
>
>Because it requires special handling between the arches and to follow the
>guidelines issued internally to Red Hat. If you have redhat-rpm-config
>installed, there are build flags that get picked up this way.

It seems like if that was the case you would want to just override the
RPM_OPT_FLAGS in those places (ie, in rpm or macro files). 

>>Also, can you use %{smp_mflags} ?
>
>why?

Indeed this is a small package and it might not be worth it, but if it works, 
why not? Would save a bit of build time. 

Agreed on everything else... 

When you have updated rawhide/cvs, let me know and reassign me on this bug, 
and I will be happy to check over changes. 


-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.

_______________________________________________
Fedora-package-review mailing list
Fedora-package-review@xxxxxxxxxx
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]