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: fribidi https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=225771 ------- Additional Comments From mtasaka@xxxxxxxxxxxxxxxxxxx 2007-02-05 05:28 EST ------- (In reply to comment #5) > (In reply to comment #4) > > > * As written in http://fedoraproject.org/wiki/Packaging/Guidelines , > > setting buildroot as > > %{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n) > > is "preferred", and also this is _not_ mandatory. > > Making to use %__id_u mandatory is still under discussion, > > some reviewers and sponsors strongly disagree this. > This has changed: last week, the FPC has decided to make it mandatory. I think this bug is not a preferred place to discuss, however, while I don't disagree (rather I recommend) to use __id_u, what I fear that the macro for _id_u is as written in /usr/lib/rpm/macros, ------------------------------------------------ # # fixowner, fixgroup, and fixperms are run at the end of hardcoded setup # These macros are necessary only for legacy compatibility, and have moved # to per-platform macro configuration (i.e. /usr/lib/rpm/<arch>-<os>/macros) # # Note: These are no longer enabled by default. #%__id_u %{__id} -u #%__chown_Rhf %{__chown} -Rhf #%__chgrp_Rhf %{__chgrp} -Rhf #%_fixowner [ `%{__id_u}` = '0' ] && %{__chown_Rhf} root #%_fixgroup [ `%{__id_u}` = '0' ] && %{__chgrp_Rhf} root #%_fixperms %{__chmod} -Rf a+rX,u+w,g-w,o-w # -------------------------------------------------- ... it seems that this is written for _legacy_ support, and these macros may be removed in the future. -- 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