[Bug 761619] Review Request: cross-binutils - Multiple cross-build binutils
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=761619 Peter Lemenkov lemen...@gmail.com changed: What|Removed |Added Status|ON_QA |CLOSED CC||lemen...@gmail.com Resolution|--- |CURRENTRELEASE Last Closed||2012-11-26 11:23:22 -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 761619] Review Request: cross-binutils - Multiple cross-build binutils
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=761619 David Howells dhowe...@redhat.com changed: What|Removed |Added AssignedTo|nob...@fedoraproject.org|dw...@infradead.org -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 761619] Review Request: cross-binutils - Multiple cross-build binutils
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=761619 --- Comment #9 from Fedora Update System upda...@fedoraproject.org 2012-03-23 08:14:18 EDT --- cross-binutils-2.22.52.0.1-8.1.fc16,cross-gcc-4.7.0-0.11.4.fc16 has been submitted as an update for Fedora 16. https://admin.fedoraproject.org/updates/cross-binutils-2.22.52.0.1-8.1.fc16,cross-gcc-4.7.0-0.11.4.fc16 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 761619] Review Request: cross-binutils - Multiple cross-build binutils
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=761619 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|NEW |MODIFIED -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 761619] Review Request: cross-binutils - Multiple cross-build binutils
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=761619 --- Comment #10 from Fedora Update System upda...@fedoraproject.org 2012-03-23 10:53:53 EDT --- cross-binutils-2.22.52.0.1-8.1.fc17,cross-gcc-4.7.0-0.11.4.fc17 has been submitted as an update for Fedora 17. https://admin.fedoraproject.org/updates/cross-binutils-2.22.52.0.1-8.1.fc17,cross-gcc-4.7.0-0.11.4.fc17 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 761619] Review Request: cross-binutils - Multiple cross-build binutils
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=761619 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #11 from Fedora Update System upda...@fedoraproject.org 2012-03-23 13:09:12 EDT --- cross-binutils-2.22.52.0.1-8.1.fc17, cross-gcc-4.7.0-0.11.4.fc17 has been pushed to the Fedora 17 testing repository. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 761619] Review Request: cross-binutils - Multiple cross-build binutils
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=761619 David Howells dhowe...@redhat.com changed: What|Removed |Added Flag||fedora-cvs? --- Comment #7 from David Howells dhowe...@redhat.com 2012-03-22 05:48:50 EDT --- New Package SCM Request === Package Name: cross-binutils Short Description: Cross-compilation binutils Owners: dhowells Branches: f16 f17 InitialCC: -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 761619] Review Request: cross-binutils - Multiple cross-build binutils
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=761619 --- Comment #8 from Jon Ciesla limburg...@gmail.com 2012-03-22 08:31:33 EDT --- Git done (by process-git-requests). David W, please take ownership of review BZs, thanks! -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 761619] Review Request: cross-binutils - Multiple cross-build binutils
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=761619 --- Comment #5 from David Howells dhowe...@redhat.com 2012-03-21 11:21:42 EDT --- Created attachment 571742 -- https://bugzilla.redhat.com/attachment.cgi?id=571742 rpmlint of the most recent SRPM and built RPM files -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 761619] Review Request: cross-binutils - Multiple cross-build binutils
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=761619 David Woodhouse dw...@infradead.org changed: What|Removed |Added Flag||fedora-review+ --- Comment #6 from David Woodhouse dw...@infradead.org 2012-03-21 12:10:39 EDT --- This looks fine now, go ahead. Thanks. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 761619] Review Request: cross-binutils - Multiple cross-build binutils
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=761619 --- Comment #4 from David Howells dhowe...@redhat.com 2012-03-15 14:04:15 EDT --- I've updated to binutils-2.22.25.0.1-8. The revised SRPM and specfile are: http://people.redhat.com/~dhowells/cross/cross-binutils-2.22.52.0.1-8.1.fc16.src.rpm http://people.redhat.com/~dhowells/cross/cross-binutils-2.22.52.0.1.spec -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 761619] Review Request: cross-binutils - Multiple cross-build binutils
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=761619 --- Comment #3 from David Howells dhowe...@redhat.com 2012-01-10 18:34:37 EST --- I've fixed the warnings that can be fixed; I have left the dangling symlink warnings as they refer are cross-package references to the common manual pages, and, after asking advice, I've left the hardlinks in. The revised SRPM can be found here: http://people.redhat.com/~dhowells/cross/cross-binutils-2.22-1.1.fc16.src.rpm I've tacked on an extra bit to the revision ID to retain the revision number of the Fedora binutils package from which I derived this whilst adding a differentiator for my own changes. The revised specfile can be found here: http://people.redhat.com/~dhowells/cross/cross-binutils.spec Note that this is the same place as the previous one, which has been renamed. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 761619] Review Request: cross-binutils - Multiple cross-build binutils
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=761619
--- Comment #2 from David Howells dhowe...@redhat.com 2011-12-22 05:23:00 EST
---
(In reply to comment #1)
You've explained the dangling symlinks, and I think that's OK.
Possibly I should just put all the manual page symlinks into the doc rpm,
though I'd prefer them only to be installed if the things they're describing
are present.
If you can arrange a man page for ld, that would be nice but it's not
mandatory.
There are manual pages for ld. Do you mean ld.bfd? If so, there is no manual
page specifically for that. I'm not entirely sure what the ld.bfd is for.
We'll have to check what the rules are for non-standard directories in /usr,
I wonder if that's going to happen if I let it create, say,
/usr/xtensa-linux-gnu/ instead of /usr/cross/xtensa-linux-gnu/. I wonder if
these things should be in /usr/libexec.
and the cross-directory hard links.
I missed that. Interesting... I wonder if the core binutils package does this
too. Even though it's marked as a cross-dir hardlink, it isn't installed so
(even though it could be).
Also:
cross-binutils.src:49: W: macro-in-comment %{version}
That's a comment borrowed from the original binutils.spec. I should probably
keep it as long as that does.
cross-binutils.src:138: W: macro-in-comment %ifarch
cross-binutils.src:139: W: macro-in-comment %if
cross-binutils.src:139: W: macro-in-comment %{_lib}
cross-binutils.src:140: W: macro-in-comment %patch03
cross-binutils.src:141: W: macro-in-comment %endif
cross-binutils.src:142: W: macro-in-comment %endif
I'm not sure whether I need this. I should find an IA64 box and try it.
cross-binutils.src:398: W: macro-in-comment %{buildroot}
cross-binutils.src:398: W: macro-in-comment %{_prefix}
cross-binutils.src:399: W: macro-in-comment %{buildroot}
cross-binutils.src:399: W: macro-in-comment %{_mandir}
I should get rid of those.
cross-binutils.src: W: patch-not-applied Patch3:
binutils-2.20.51.0.2-ia64-lib64.patch
That's one of the original binutils.spec patches and is related to the
macro-in-comment warnings of lines 138-142. I wonder if I should just apply
all of the original binutils.spec patches as applied by that?
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 761619] Review Request: cross-binutils - Multiple cross-build binutils
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=761619
--- Comment #1 from David Woodhouse dw...@infradead.org 2011-12-19 16:37:41
EST ---
binutils-xtensa-linux-gnu.x86_64: W: cross-directory-hard-link
/usr/cross/xtensa-linux-gnu/bin/ranlib /usr/bin/xtensa-linux-gnu-ranlib
binutils-xtensa-linux-gnu.x86_64: W: dangling-relative-symlink
/usr/share/man/man1/xtensa-linux-gnu-c++filt.1.gz cross-c++filt.1.gz
binutils-xtensa-linux-gnu.x86_64: W: no-manual-page-for-binary
xtensa-linux-gnu-ld.bfd
binutils-xtensa-linux-gnu.x86_64: W: non-standard-dir-in-usr cross
You've explained the dangling symlinks, and I think that's OK. If you can
arrange a man page for ld, that would be nice but it's not mandatory.
We'll have to check what the rules are for non-standard directories in /usr,
and the cross-directory hard links.
Also:
cross-binutils.src:49: W: macro-in-comment %{version}
cross-binutils.src:138: W: macro-in-comment %ifarch
cross-binutils.src:139: W: macro-in-comment %if
cross-binutils.src:139: W: macro-in-comment %{_lib}
cross-binutils.src:140: W: macro-in-comment %patch03
cross-binutils.src:141: W: macro-in-comment %endif
cross-binutils.src:142: W: macro-in-comment %endif
cross-binutils.src:398: W: macro-in-comment %{buildroot}
cross-binutils.src:398: W: macro-in-comment %{_prefix}
cross-binutils.src:399: W: macro-in-comment %{buildroot}
cross-binutils.src:399: W: macro-in-comment %{_mandir}
cross-binutils.src: W: patch-not-applied Patch3:
binutils-2.20.51.0.2-ia64-lib64.patch
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 761619] Review Request: cross-binutils - Multiple cross-build binutils
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=761619 David Howells dhowe...@redhat.com changed: What|Removed |Added Blocks||766166 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
