[Bug 814916] Review Request: sratom - a C library for serializing LV2 plugins
https://bugzilla.redhat.com/show_bug.cgi?id=814916 Fedora Update System changed: What|Removed |Added Status|ON_QA |CLOSED Resolution|--- |ERRATA Last Closed||2012-05-27 21:22:13 --- Comment #13 from Fedora Update System --- sratom-0.2.0-3.fc17 has been pushed to the Fedora 17 stable repository. -- 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 814916] Review Request: sratom - a C library for serializing LV2 plugins
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=814916 Fedora Update System changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #12 from Fedora Update System 2012-05-15 12:41:56 EDT --- sratom-0.2.0-3.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 814916] Review Request: sratom - a C library for serializing LV2 plugins
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=814916 --- Comment #11 from Fedora Update System 2012-05-15 01:55:06 EDT --- sratom-0.2.0-3.fc17 has been submitted as an update for Fedora 17. https://admin.fedoraproject.org/updates/sratom-0.2.0-3.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 814916] Review Request: sratom - a C library for serializing LV2 plugins
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=814916 Fedora Update System changed: What|Removed |Added Status|ASSIGNED|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 814916] Review Request: sratom - a C library for serializing LV2 plugins
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=814916 --- Comment #10 from Jon Ciesla 2012-04-29 17:00:03 EDT --- Git done (by process-git-requests). -- 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 814916] Review Request: sratom - a C library for serializing LV2 plugins
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=814916 Brendan Jones changed: What|Removed |Added Flag||fedora-cvs? --- Comment #9 from Brendan Jones 2012-04-29 15:25:56 EDT --- Thanks for taking this review New Package SCM Request === Package Name: sratom Short Description: A C library for serializing LV2 atoms Owners: bsjones 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 814916] Review Request: sratom - a C library for serializing LV2 plugins
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=814916 Mattias Ellert changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Comment #8 from Mattias Ellert 2012-04-24 11:22:28 EDT --- Package approved. -- 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 814916] Review Request: sratom - a C library for serializing LV2 plugins
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=814916 --- Comment #7 from Brendan Jones 2012-04-23 07:46:34 EDT --- SPEC: http://bsjones.fedorapeople.org/lv2/sratom.spec SRPM: http://bsjones.fedorapeople.org/lv2/sratom-0.2.0-2.fc16.src.rpm I've addressed the issues you picked up. Apologies for the missing BR. I've discovered a bug in the build tests for i686, and this will currently fail in mock. Seems to be OK for x86_64. I'll work through this with upstream and get it patched. -- 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 814916] Review Request: sratom - a C library for serializing LV2 plugins
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=814916 Mattias Ellert changed: What|Removed |Added Flag||fedora-review? --- Comment #6 from Mattias Ellert 2012-04-22 15:40:58 EDT --- Fedora Review sratom 2012-04-22 $ rpmlint sratom*.rpm sratom.src: W: spelling-error Summary(en_US) serialising -> serializing, serialization sratom.src: W: spelling-error %description -l en_US serialising -> serializing, serialization sratom.src: W: spelling-error %description -l en_US serialisation -> serialization, sensationalist, sensationalism sratom.src: W: spelling-error %description -l en_US serialise -> serialize, serial, aerialist sratom.x86_64: W: spelling-error Summary(en_US) serialising -> serializing, serialization sratom.x86_64: W: spelling-error %description -l en_US serialising -> serializing, serialization sratom.x86_64: W: spelling-error %description -l en_US serialisation -> serialization, sensationalist, sensationalism sratom.x86_64: W: spelling-error %description -l en_US serialise -> serialize, serial, aerialist sratom-devel.x86_64: W: spelling-error %description -l en_US serialising -> serializing, serialization sratom-devel.x86_64: W: spelling-error %description -l en_US serialisation -> serialization, sensationalist, sensationalism sratom-devel.x86_64: W: spelling-error %description -l en_US serialise -> serialize, serial, aerialist 4 packages and 0 specfiles checked; 0 errors, 11 warnings. Clean except for American English spelling in Summary and Description. + Package name according to guidelines + Specfile named after package + Package license (MIT) Fedora approved + Package license reflects license as stated in the sources + License file (COPYING) included as %doc - Specfile uses British spelling in summary and description This should be changed to American spelling according to guidelines See also the rpmlint report above + Specfile is readable + Source matches upstream $ md5sum sratom-0.2.0.tar.bz2 srpm/sratom-0.2.0.tar.bz2 c03cf2849186818610ffe889be4f5b55 sratom-0.2.0.tar.bz2 c03cf2849186818610ffe889be4f5b55 srpm/sratom-0.2.0.tar.bz2 - Package fail to build due to missing BuildRequires: python ./waf configure -v --prefix=/usr --libdir=/usr/lib64 --mandir=/usr/share/man --datadir=/usr/share --docdir=/usr/share/doc/sratom-devel-0.2.0 --test --docs /usr/bin/env: python: No such file or directory error: Bad exit status from /var/tmp/rpm-tmp.3g1rbM (%build) [ Also my rebuild of the lv2 package on which this package depends failed with the same error ] + No locales + Scriptlets call ldconfig + No bundled libraries + Package owns directories it creates + No duplicates in %files + File permissions are sane + Specfile uses macros consistently + Contains code + Documentation is in -devel, though can be considered small + %doc not runtime essential + No static libraries + Headers and develoment library symlinks in -devel + -devel requires main with fully qualified version + No .la files + Package does not own ohters' libraries + Filenames valid UTF8 Summary: Fix American spelling and add missing BR on python -- 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 814916] Review Request: sratom - a C library for serializing LV2 plugins
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=814916 Orcan Ogetbil changed: What|Removed |Added Depends on||814542 --- Comment #5 from Orcan Ogetbil 2012-04-22 11:43:11 EDT --- I made a full review on this, but I see that Mattias is already assigned to the bug. I'll post my findings anyways: ! rpmlint says: sratom.x86_64: W: spelling-error %description -l en_US serialisation -> serialization, sensationalist, sensationalism sratom.x86_64: W: spelling-error %description -l en_US serialise -> serialize, serial, aerialist sratom.src: W: spelling-error Summary(en_US) serialising -> serializing, serialization There was a requirement that says the spec file must be written in American English, so I guess we should use serializ* * Need to add BuildRequires: serd-devel >= 0.14.0 Other than the above, the package is good to go. -- 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 814916] Review Request: sratom - a C library for serializing LV2 plugins
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=814916 --- Comment #4 from Brendan Jones 2012-04-22 02:45:44 EDT --- That's right - not specifically, I just wanted to hold up the qtractor review against the new LV2 (lv2/lilv/suil) updates. Up until now qtractor is the only package that is using suil/lilv. -- 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 814916] Review Request: sratom - a C library for serializing LV2 plugins
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=814916 --- Comment #3 from Orcan Ogetbil 2012-04-21 23:44:55 EDT --- I see that the new lilv requires sratom, but I don't think qtractor requires the new lilv specifically. -- 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 814916] Review Request: sratom - a C library for serializing LV2 plugins
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=814916 Orcan Ogetbil changed: What|Removed |Added CC||oget.fed...@gmail.com --- Comment #2 from Orcan Ogetbil 2012-04-21 21:00:43 EDT --- Hi, could you clarify why this blocks qtractor? -- 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 814916] Review Request: sratom - a C library for serializing LV2 plugins
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=814916 --- Comment #1 from Brendan Jones 2012-04-21 17:19:47 EDT --- Build requirements exist in rawhide only for the moment -- 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 814916] Review Request: sratom - a C library for serializing LV2 plugins
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=814916 Brendan Jones changed: What|Removed |Added Version|16 |rawhide -- 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 814916] Review Request: sratom - a C library for serializing LV2 plugins
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=814916 Mattias Ellert changed: What|Removed |Added Status|NEW |ASSIGNED CC||mattias.ell...@fysast.uu.se AssignedTo|nob...@fedoraproject.org|mattias.ell...@fysast.uu.se -- 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 814916] Review Request: sratom - a C library for serializing LV2 plugins
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=814916 Brendan Jones changed: What|Removed |Added Blocks||805236(FedoraAudio), 814924 -- 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 814916] Review Request: sratom - a C library for serializing LV2 plugins
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=814916 Brendan Jones changed: What|Removed |Added Blocks||797418 -- 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