Note: This is a public test instance of Red Hat Bugzilla. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback at bugzilla.redhat.com.
Bug 170978
Summary: | Review Request: nomadsync | ||
---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Linus Walleij <triad> |
Component: | Package Review | Assignee: | Michael A. Peters <mpeters> |
Status: | CLOSED NEXTRELEASE | QA Contact: | David Lawrence <dkl> |
Severity: | medium | Docs Contact: | |
Priority: | medium | ||
Version: | rawhide | CC: | bnocera, fedora-extras-list |
Target Milestone: | --- | ||
Target Release: | --- | ||
Hardware: | All | ||
OS: | Linux | ||
URL: | http://nomadsync.sourceforge.net/ | ||
Whiteboard: | |||
Fixed In Version: | Doc Type: | Bug Fix | |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2006-03-15 22:13:20 UTC | Type: | --- |
Regression: | --- | Mount Type: | --- |
Documentation: | --- | CRM: | |
Verified Versions: | Category: | --- | |
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
Cloudforms Team: | --- | Target Upstream Version: | |
Embargoed: | |||
Bug Depends On: | |||
Bug Blocks: | 163779 |
Description
Linus Walleij
2005-10-16 20:19:03 UTC
Is id3lib already shipped in the extras? Also, you could simply have your .desktop file as another Source: Source1: nomadsync.desktop (In reply to comment #1) > Is id3lib already shipped in the extras? Yes. It's not id3lib it's libid3tag (there is also taglib and libmetatag somehwere just to add to your confusion) and yes, that is already in the extras too. I switched to using a Source1: line instead of a patch, much smarter, indeed. Fixed package: Spec Name or Url: http://www.df.lth.se/~triad/krad/fc/nomadsync.spec SRPM Name or Url: http://www.df.lth.se/~triad/krad/fc/nomadsync-0.4.2-4.src.rpm EH sorry yes, no its id3lib not libid3tag and yes as pointed out its in the extras already. I was drunk... or rather... well. In %install section - please change %makeinstall to make DESTDIR=%buildroot install I don't know if it is issue here, but I've had issue with the former before that don't happen with the latter (strings including the buildroot in files) I tested the change - it works. %defattr(-, root, root) please change to %defattr(-,root,root,-) rpmlint errors (after mock build): [mpeters@utility result]$ rpmlint *.rpm E: nomadsync script-without-shellbang /usr/share/doc/nomadsync-0.4.2/copying E: nomadsync script-without-shellbang /usr/share/doc/nomadsync-0.4.2/ChangeLog E: nomadsync wrong-script-end-of-line-encoding /usr/share/doc/nomadsync-0.4.2/ChangeLog E: nomadsync script-without-shellbang /usr/share/doc/nomadsync-0.4.2/install E: nomadsync script-without-shellbang /usr/share/doc/nomadsync-0.4.2/readme E: nomadsync wrong-script-end-of-line-encoding /usr/share/doc/nomadsync-0.4.2/readme E: nomadsync script-without-shellbang /usr/share/doc/nomadsync-0.4.2/authors [mpeters@utility result]$ The "script-without-shellbang" errors can be fixed by removing execution bit on those files. The "wrong-script-end-of-line-encoding" errors can be fixed with sed. In %prep after extraction do something like: %{__sed} -i 's?\r??' ChangeLog %{__sed} -i 's?\r??' readme -=- Please don't %doc the install file. It is already installed once the user installs the rpm. I fixed these things, but no cigar, because now it doesn't compile under GCC 4.0.2 so more things need to be done... Hm the package is set into review, but who reviews it? It is still assigned to Greg, which is also default. Greg, is it really you who review this package? I think I set it to review and forgot to set it to me. OK thanks Michael, I'll get back as soon as I find a proper patch for GCC 4.0.2. Fixed package: Spec Name or Url: http://www.df.lth.se/~triad/krad/fc/nomadsync.spec SRPM Name or Url: http://www.df.lth.se/~triad/krad/fc/nomadsync-0.4.2-5.src.rpm Turns out there was no compiler issue, it was the unicodification of wxGTK 2.6 that was the culprit, so this requires 2.4.x. I saw that package was pulled from Extras anyway... Will need some compat package in devel I guess? I figure I will replace wxGTK-devel for compat-wxGTK or compat-wxGTK2 in the devel (FC5) version. Do you want me to provide a separate spec for devel? That would be best. Fixed package: Spec Name or Url: http://www.df.lth.se/~triad/krad/fc/nomadsync.spec SRPM Name or Url: http://www.df.lth.se/~triad/krad/fc/nomadsync-0.4.2-6.src.rpm I actually detect FC version instead since it's a oneliner. Doesn't hurt to have a generic specfile in this case. Would it better to simply BuildRequires %{_bindir}/wxgtk-2.4-config rather than fedora specific macros ?? How does the 64-bit package for wxgtk-devel handle that? (can 32 and 64 bit be || installed?) Yes, that works much better. Sorry for fat delay on this, I missed it somehow: Spec Name or Url: http://www.df.lth.se/~triad/krad/fc/nomadsync.spec SRPM Name or Url: http://www.df.lth.se/~triad/krad/fc/nomadsync-0.4.2-7.src.rpm Good: * Builds fine in mock (fc4 i386) * rpmlint clean * owns all directories it installs * proper rpm Group * clean rpm spec file * proper summary and description in American English * spec file understabdable * proper and consistent use of macros * proper installation of Desktop file, with X-Fedora category and fedora vendor * proper permissions on files BAD md5sum in src.rpm: f8d50c14fae3f8cdedc57c491c7d65d4 ../SOURCES/nomadsync-0.4.2.tar.gz md5sum upstream: fd55927c54cd0737cdecb88e4bfbf0b7 nomadsync-0.4.2.tar.gz -=- looking at the two - nomadsync is upstream source unpacked, nomadsync-0.4.2.src is from the src.rpm [mpeters@jerusalem delete_me]$ diff -u nomadsync nomadsync-0.4.2.src Only in nomadsync-0.4.2.src: Makefile Only in nomadsync-0.4.2.src: config.log Only in nomadsync-0.4.2.src: config.status Common subdirectories: nomadsync/cvs and nomadsync-0.4.2.src/cvs Common subdirectories: nomadsync/debug and nomadsync-0.4.2.src/debug Common subdirectories: nomadsync/doc and nomadsync-0.4.2.src/doc Only in nomadsync-0.4.2.src: libtool Common subdirectories: nomadsync/macros and nomadsync-0.4.2.src/macros Common subdirectories: nomadsync/optimized and nomadsync-0.4.2.src/optimized Common subdirectories: nomadsync/src and nomadsync-0.4.2.src/src Common subdirectories: nomadsync/templates and nomadsync-0.4.2.src/templates [mpeters@jerusalem delete_me]$ It looks like the source tarball you have had configure run in it before it was archived. Use the pristine upstream source tarball. Not strange since the upstream source was updated today (file dated 2005-03-08), updated to latest upstream: Spec Name or Url: http://www.df.lth.se/~triad/krad/fc/nomadsync.spec SRPM Name or Url: http://www.df.lth.se/~triad/krad/fc/nomadsync-0.4.2-8.src.rpm Hey yeah - you're right. Odd they did not version tarball ... md5sum matches now. approved. OK (after some patching on devel) it now builds on all archs. Michael: THANK YOU for reviewing this RPM! |