[Bug 600243] Review Request: libjpeg-turbo - MMX/SSE accelerated libjpeg
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=600243 --- Comment #52 from Rex Dieter 2010-06-28 13:21:31 EDT --- either would be fine for testing purposes, but I suppose adding it to -utils (to match the Obsoletes) would be more appropriate -- 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 600243] Review Request: libjpeg-turbo - MMX/SSE accelerated libjpeg
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=600243 --- Comment #51 from Chen Lei 2010-06-28 12:42:53 EDT --- (In reply to comment #50) > Your way is a little harder and could lead to more churn. If you and the > maintainers involved don't mind that... so be it. Then, which way will be more appropriate? Adding Provides: libjpeg = 6b to libjpeg-turbo or libjpeg-turbo-utils? -- 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 600243] Review Request: libjpeg-turbo - MMX/SSE accelerated libjpeg
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=600243 --- Comment #50 from Rex Dieter 2010-06-28 11:03:10 EDT --- Your way is a little harder and could lead to more churn. If you and the maintainers involved don't mind that... so be it. -- 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 600243] Review Request: libjpeg-turbo - MMX/SSE accelerated libjpeg
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=600243 --- Comment #49 from Chen Lei 2010-06-28 10:35:24 EDT --- (In reply to comment #48) > I'll respectfully disagree with comment #46 ( and comment #6 ) and argue that > the Provides should be included, at least temporarily while testing is > underway. > Else, it will be impossible to truly test swapping between > libjpeg/libjpeg-turbo without breaking dependencies. Hi Rex, The situation is bit complicated, libjpeg is split into libjpeg-turbo and libjpeg-turbo-utils now. 5 packages will be broken after building libjpeg-turbo in koji, if we provide libjpeg in libjpeg-turbo, then we'll break some packages silently because they actually depend on libjpeg-turbo-utils instead of linking with libjpeg. Providing libjpeg in libjpeg-turbo-utils can solve all broken dependecies, but the result will be a bit strange when we type "yum install libjpeg". 5 packages is not a large amount, also we only need to fix openjdk once, rpmbuild already add libjpeg.so.62 dependency to openjdk automatically, we don't need to do it again if fedora 14 revert to use libjpeg. repoquery --whatrequires --exactdeps libjpeg libjpeg-0:6b-46.fc12.x86_64 libjpeg-0:6b-46.fc12.i686 gallery2-jpegtran-0:2.3.1-1.fc13.noarch java-1.6.0-openjdk-1:1.6.0.0-37.b17.fc13.x86_64 gocr-0:0.48-1.fc13.x86_64 darkplaces-0:20091001-2.fc13.x86_64 libjpeg-devel-0:6b-46.fc12.x86_64 libjpeg-devel-0:6b-46.fc12.i686 renrot-0:1.1-1.fc13.2.noarch java-1.6.0-openjdk-1:1.6.0.0-39.b18.fc13.x86_64 -- 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 600243] Review Request: libjpeg-turbo - MMX/SSE accelerated libjpeg
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=600243 --- Comment #48 from Rex Dieter 2010-06-28 10:12:54 EDT --- I'll respectfully disagree with comment #46 ( and comment #6 ) and argue that the Provides should be included, at least temporarily while testing is underway. Else, it will be impossible to truly test swapping between libjpeg/libjpeg-turbo without breaking dependencies. -- 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 600243] Review Request: libjpeg-turbo - MMX/SSE accelerated libjpeg
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=600243 Adam Tkac changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||NEXTRELEASE --- Comment #47 from Adam Tkac 2010-06-28 08:57:02 EDT --- libjpeg-turbo-0.0.93-9.fc14 has been successfully built, thanks for the review. Closing. -- 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 600243] Review Request: libjpeg-turbo - MMX/SSE accelerated libjpeg
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=600243 --- Comment #46 from Adam Tkac 2010-06-28 08:55:29 EDT --- (In reply to comment #43) > Comment #40 could be mitigated if libjpeg-turbo also included: > Provides: libjpeg = 6b-47 This is not needed, as explained in comment #6. -- 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 600243] Review Request: libjpeg-turbo - MMX/SSE accelerated libjpeg
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=600243 --- Comment #45 from Jason Tibbitts 2010-06-26 03:56:28 EDT --- CVS done (by process-cvs-requests.py). -- 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 600243] Review Request: libjpeg-turbo - MMX/SSE accelerated libjpeg
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=600243 --- Comment #44 from Rex Dieter 2010-06-25 12:52:20 EDT --- oh, and you're doing that, may as well also Provides: libjpeg%{?_isa} = 6b-47 -- 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 600243] Review Request: libjpeg-turbo - MMX/SSE accelerated libjpeg
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=600243 Rex Dieter changed: What|Removed |Added CC||rdie...@math.unl.edu --- Comment #43 from Rex Dieter 2010-06-25 12:48:40 EDT --- Comment #40 could be mitigated if libjpeg-turbo also included: Provides: libjpeg = 6b-47 -- 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 600243] Review Request: libjpeg-turbo - MMX/SSE accelerated libjpeg
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=600243 Adam Tkac changed: What|Removed |Added Flag||fedora-cvs? --- Comment #42 from Adam Tkac 2010-06-24 07:22:20 EDT --- New Package CVS Request === Package Name: libjpeg-turbo Short Description: A MMX/SSE2 accelerated library for manipulating JPEG image files Owners: atkac Branches: devel 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 600243] Review Request: libjpeg-turbo - MMX/SSE accelerated libjpeg
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=600243 --- Comment #41 from Adam Tkac 2010-06-24 07:20:26 EDT --- (In reply to comment #40) > Okay, this package is approved. Thanks for the review. > 1.Before importing libjpeg-turbo to cvs, you need to notice java-1.6.0-openjdk > maintainer to remove explict libjpeg dependency first. > > rpm -qR java-1.6.0-openjdk|grep jpeg > libjpeg = 6b > libjpeg.so.62()(64bit) I just filled bug #607554. > 2.Some trival improvement: > It will be better to move wizard.doc from -devel to -utils, > > %package utils > Summary: Tools for manipulating JPEG images > -> > %package utils > Summary: Utilities for manipulating JPEG images Ok, I will incorporate your suggestions to the first build of libjpeg-turbo. -- 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 600243] Review Request: libjpeg-turbo - MMX/SSE accelerated libjpeg
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=600243 Chen Lei changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Comment #40 from Chen Lei 2010-06-16 23:23:38 EDT --- Okay, this package is approved. 1.Before importing libjpeg-turbo to cvs, you need to notice java-1.6.0-openjdk maintainer to remove explict libjpeg dependency first. rpm -qR java-1.6.0-openjdk|grep jpeg libjpeg = 6b libjpeg.so.62()(64bit) 2.Some trival improvement: It will be better to move wizard.doc from -devel to -utils, %package utils Summary: Tools for manipulating JPEG images -> %package utils Summary: Utilities for manipulating JPEG images -- 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 600243] Review Request: libjpeg-turbo - MMX/SSE accelerated libjpeg
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=600243 --- Comment #39 from Adam Tkac 2010-06-16 12:41:12 EDT --- (In reply to comment #27) > Issues: > 1. BR: nasm should be conditional > 2. It'll be better to add more files to %doc > 3. I suggest you to change -tools subpackage to -utils I incorporated all your suggestions to the new srpm + spec: http://atkac.fedorapeople.org/libjpeg-turbo/libjpeg-turbo-0.0.93-8.fc14.src.rpm http://atkac.fedorapeople.org/libjpeg-turbo/libjpeg-turbo.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 600243] Review Request: libjpeg-turbo - MMX/SSE accelerated libjpeg
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=600243 --- Comment #38 from Chen Lei 2010-06-16 07:57:25 EDT --- (In reply to comment #37) > Some people are actually using rawhide distribution. You don't need to > deliberately create broken deps if it's easy to avoid that. > First create a compatible package, then fix up everything that depends on the > compatibility interface, and only after everything is fixed remove the > compatibility interface. > The libjpeg-turbo feature page states that no rebuilds are needed to switch to > libjpeg-turbo as it is a drop-in replacement. You are now suggesting to make > it > non-drop-in, why would that be good? Also, the feature page says there is a > contingency plan to revert back to libjpeg if there are serious problems. > Chan, > do you understand that if you quickly rebuild all packages to depend on > libjpeg-turbo directly, then it will take another rebuild to revert back to > libjpeg if it turns out libjpeg-turbo isn't working out? Only 5 packages need fix, among those packages only openjdk need a quick fix. For openjdk, the fix is simply removing explict depencenies on libjpeg, we don't need another rebuild if we revert back to libjpeg. "yum install libjpeg" will be silly if we add provides libjpeg to libjpeg-turbo-tools. I don't want to discuss this issue again unless fedora packaging guideline recommends to do so. -- 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 600243] Review Request: libjpeg-turbo - MMX/SSE accelerated libjpeg
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=600243 --- Comment #37 from Kalev Lember 2010-06-16 07:42:04 EDT --- Some people are actually using rawhide distribution. You don't need to deliberately create broken deps if it's easy to avoid that. First create a compatible package, then fix up everything that depends on the compatibility interface, and only after everything is fixed remove the compatibility interface. The libjpeg-turbo feature page states that no rebuilds are needed to switch to libjpeg-turbo as it is a drop-in replacement. You are now suggesting to make it non-drop-in, why would that be good? Also, the feature page says there is a contingency plan to revert back to libjpeg if there are serious problems. Chan, do you understand that if you quickly rebuild all packages to depend on libjpeg-turbo directly, then it will take another rebuild to revert back to libjpeg if it turns out libjpeg-turbo isn't working out? -- 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 600243] Review Request: libjpeg-turbo - MMX/SSE accelerated libjpeg
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=600243 --- Comment #36 from Chen Lei 2010-06-16 07:22:46 EDT --- (In reply to comment #35) > The Provides: libjpeg is needed to provide a clean upgrade path for the list > of > packages in comment #28 which have Requires: libjpeg. Upgrade path only applies to end users, rawhide is a place which allow to have broken denpendecies, one week is enough to treat all those broken dependencies. -- 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 600243] Review Request: libjpeg-turbo - MMX/SSE accelerated libjpeg
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=600243 --- Comment #35 from Kalev Lember 2010-06-16 06:56:19 EDT --- The Provides: libjpeg is needed to provide a clean upgrade path for the list of packages in comment #28 which have Requires: libjpeg. -- 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 600243] Review Request: libjpeg-turbo - MMX/SSE accelerated libjpeg
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=600243 --- Comment #34 from Chen Lei 2010-06-15 23:36:33 EDT --- (In reply to comment #33) > As soname deps are automatically handled by rpm, I bet most of the packages > which currently have manual libjpeg Requires do that because they really need > one of the tools. That's also why it's libjpeg-turbo-tools that is Obsoleting > libjpeg package, as opposed to libjpeg-turbo having the Obsoletes. > > Since the libjpeg-turbo-tools package already contains Obsoletes: libjpeg, > it'd > make sense to add the Provides: libjpeg also to the same libjpeg-turbo-tools > package. > > Right now -tools subpackage has: > Obsoletes: libjpeg < 6b-47 > > To provide a clean upgrade path it should be: > Provides: libjpeg = 6b-47 > Obsoletes: libjpeg < 6b-47 > > > (In reply to comment #32) > > Also provides libjpeg in libjpeg-turbo will violate packaging guideline. > > Huh, how so? In fact, packaging guidelines [1] suggest to use the following > scheme to replace an existing package: > Provides: oldpackagename = $provEVR > Obsoletes: oldpackagename < $obsEVR > > [1] > http://fedoraproject.org/wiki/PackageNamingGuidelines#Renaming.2Freplacing_existing_packages > I already explained this to Adam. See comments 6 and https://fedoraproject.org/wiki/Upgrade_paths_—_renaming_or_splitting_packages -- 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 600243] Review Request: libjpeg-turbo - MMX/SSE accelerated libjpeg
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=600243 --- Comment #33 from Kalev Lember 2010-06-15 23:28:28 EDT --- As soname deps are automatically handled by rpm, I bet most of the packages which currently have manual libjpeg Requires do that because they really need one of the tools. That's also why it's libjpeg-turbo-tools that is Obsoleting libjpeg package, as opposed to libjpeg-turbo having the Obsoletes. Since the libjpeg-turbo-tools package already contains Obsoletes: libjpeg, it'd make sense to add the Provides: libjpeg also to the same libjpeg-turbo-tools package. Right now -tools subpackage has: Obsoletes: libjpeg < 6b-47 To provide a clean upgrade path it should be: Provides: libjpeg = 6b-47 Obsoletes: libjpeg < 6b-47 (In reply to comment #32) > Also provides libjpeg in libjpeg-turbo will violate packaging guideline. Huh, how so? In fact, packaging guidelines [1] suggest to use the following scheme to replace an existing package: Provides: oldpackagename = $provEVR Obsoletes: oldpackagename < $obsEVR [1] http://fedoraproject.org/wiki/PackageNamingGuidelines#Renaming.2Freplacing_existing_packages -- 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 600243] Review Request: libjpeg-turbo - MMX/SSE accelerated libjpeg
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=600243 --- Comment #32 from Chen Lei 2010-06-15 22:45:13 EDT --- (In reply to comment #31) > Right now -tools subpackages has Obsoletes: libjpeg < 6b-47, so it'd make > sense > to add the Provides also to -tools subpackage. Provides what? Original libjpeg don't have a -tools subpackage, also it's easy to fix those broken dependencies, I don't think it's a real issue for all of us. Also provides libjpeg in libjpeg-turbo will violate packaging guideline. -- 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 600243] Review Request: libjpeg-turbo - MMX/SSE accelerated libjpeg
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=600243 --- Comment #31 from Kalev Lember 2010-06-15 22:25:29 EDT --- (In reply to comment #30) > (In reply to comment #29) > > (In reply to comment #28) > > > Packages need fix after review, most of those packages actually don't need > > > depend on libjpeg explicitly: > > > > To fix upgrade path for those packages I'd suggest to add Provides: libjpeg > > = > > %{version}-%{release} > > It will break some packages silently, some of those packages depend on libjpeg > utilities which now packages as a subpackage. Among those packages, only > java-1.6.0-openjdk should fix intermediatly. Right now -tools subpackages has Obsoletes: libjpeg < 6b-47, so it'd make sense to add the Provides also to -tools subpackage. -- 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 600243] Review Request: libjpeg-turbo - MMX/SSE accelerated libjpeg
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=600243 --- Comment #30 from Chen Lei 2010-06-15 22:17:01 EDT --- (In reply to comment #29) > (In reply to comment #28) > > Packages need fix after review, most of those packages actually don't need > > depend on libjpeg explicitly: > > To fix upgrade path for those packages I'd suggest to add Provides: libjpeg = > %{version}-%{release} It will break some packages silently, some of those packages depend on libjpeg utilities which now packages as a subpackage. Among those packages, only java-1.6.0-openjdk should fix intermediatly. -- 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 600243] Review Request: libjpeg-turbo - MMX/SSE accelerated libjpeg
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=600243 --- Comment #29 from Kalev Lember 2010-06-15 21:36:13 EDT --- (In reply to comment #28) > Packages need fix after review, most of those packages actually don't need > depend on libjpeg explicitly: To fix upgrade path for those packages I'd suggest to add Provides: libjpeg = %{version}-%{release} -- 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 600243] Review Request: libjpeg-turbo - MMX/SSE accelerated libjpeg
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=600243 --- Comment #28 from Chen Lei 2010-06-15 09:28:17 EDT --- Packages need fix after review, most of those packages actually don't need depend on libjpeg explicitly: repoquery --whatrequires --exactdeps libjpeg libjpeg-0:6b-46.fc12.x86_64 libjpeg-0:6b-46.fc12.i686 gallery2-jpegtran-0:2.3.1-1.fc13.noarch java-1.6.0-openjdk-1:1.6.0.0-37.b17.fc13.x86_64 gocr-0:0.48-1.fc13.x86_64 darkplaces-0:20091001-2.fc13.x86_64 libjpeg-devel-0:6b-46.fc12.x86_64 libjpeg-devel-0:6b-46.fc12.i686 renrot-0:1.1-1.fc13.2.noarch java-1.6.0-openjdk-1:1.6.0.0-39.b18.fc13.x86_64 -- 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 600243] Review Request: libjpeg-turbo - MMX/SSE accelerated libjpeg
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=600243 --- Comment #27 from Chen Lei 2010-06-15 09:09:49 EDT --- formal review here: +:ok, =:needs attention, -:needs fixing MUST Items: [+] MUST: rpmlint must be run on every package. libjpeg-turbo.x86_64: W: shared-lib-calls-exit /usr/lib64/libjpeg.so.62.0.0 e...@glibc_2.2.5 libjpeg-turbo-devel.x86_64: W: obsolete-not-provided libjpeg-static libjpeg-turbo-tools.x86_64: W: obsolete-not-provided libjpeg Those warnings are harmless, and won't fix. [+] MUST: The package must be named according to the Package Naming Guidelines. [+] MUST: The spec file name must match the base package %{name} [+] MUST: The package must meet the Packaging Guidelines. [FIXME?: covers this list and more] [+] MUST: The package must be licensed with a Fedora approved license and meet the Licensing Guidelines. [+] MUST: The License field in the package spec file must match the actual license. [+] MUST: If (and only if) the source package includes the text of the license(s) in its own file, then that file, containing the text of the license(s) for the package must be included in %doc. [+] MUST: The spec file must be written in American English. [+] MUST: The spec file for the package MUST be legible. [+] MUST: The sources used to build the package must match the upstream source, as provided in the spec URL. <>ac8bb8b00558b077c159a2f35dc196a0 [+] MUST: The package must successfully compile and build into binary rpms on at least one supported architecture. [=] MUST: All build dependencies must be listed in BuildRequires [+] MUST: The spec file MUST handle locales properly. This is done by using the %find_lang macro. [+] MUST: Every binary RPM package which stores shared library files (not just symlinks) in any of the dynamic linker's default paths, must call ldconfig in %post and %postun. [+] MUST: A package must own all directories that it creates. If it does not create a directory that it uses, then it should require a package which does create that directory. [+] MUST: A package must not contain any duplicate files in the %files listing. [+] MUST: Permissions on files must be set properly. Executables should be set with executable permissions, for example. Every %files section must include a %defattr(...) line. [+] MUST: Each package must have a %clean section, which contains rm -rf %{buildroot} (or $RPM_BUILD_ROOT). [+] MUST: Each package must consistently use macros, as described in the macros section of Packaging Guidelines. [+] MUST: The package must contain code, or permissible content. This is described in detail in the code vs. content section of Packaging Guidelines. [+] MUST: If a package includes something as %doc, it must not affect the runtime of the application. [+] MUST: Packages must NOT contain any .la libtool archives, these should be removed in the spec. [+] MUST: Packages containing GUI applications must include a %{name}.desktop file, and that file must be properly installed with desktop-file-install in the %install section. [+] MUST: Packages must not own files or directories already owned by other packages. [+] MUST: All filenames in rpm packages must be valid UTF-8. SHOULD Items: [+] SHOULD: If the source package does not include license text(s) as a separate file from upstream, the packager SHOULD query upstream to include it. [=] SHOULD: The description and summary sections in the package spec file should contain translations for supported Non-English languages, if available. [+] SHOULD: The reviewer should test that the package builds in mock. [+] SHOULD: The package should compile and build into binary rpms on all supported architectures. [+] SHOULD: The reviewer should test that the package functions as described. [+] SHOULD: If scriptlets are used, those scriptlets must be sane. [+] SHOULD: If the package has file dependencies outside of /etc, /bin, /sbin, /usr/bin, or /usr/sbin consider requiring the package which provides the file instead of the file itself. Issues: 1. BR: nasm should be conditional %ifarch %{ix86} x86_64 BuildRequires: nasm %endif 2. It'll be better to add more files to %doc %doc README README-turbo.txt change.log ChangeLog.txt LGPL.txt LICENSE.txt %file devel %doc coderules.doc jconfig.doc libjpeg.doc structure.doc wizard.doc example.c %file tools usage.doc 3. Personally, I suggest you to change -tools subpackage to -utils subpackages for two reasons: yum list \*-utils and yum list \*-tools show we have much more utils subpackages in repo; Upstream and wikipedia call those programs as utilities, the word utility will be more appropriate compared to tool. See http://en.wikipedia.org/wiki/Libjpeg Utilities The following utility programs are available with libjpeg: -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC
[Bug 600243] Review Request: libjpeg-turbo - MMX/SSE accelerated libjpeg
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=600243 --- Comment #26 from Adam Tkac 2010-06-14 07:29:18 EDT --- (In reply to comment #25) > How about to ask this on -devel? I found that many core libraries in fedora > provides static libs, e.g. glibc openssl zlib libpng. Done. Check http://lists.fedoraproject.org/pipermail/devel/2010-June/137604.html. > If we don't provide -static subpackage, we need to obsolete it in the main > package. Right you are but it should be obsoleted in the -devel subpackage. New spec + SRPM: http://atkac.fedorapeople.org/libjpeg-turbo/libjpeg-turbo.spec http://atkac.fedorapeople.org/libjpeg-turbo/libjpeg-turbo-0.0.93-7.fc14.src.rpm -- 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 600243] Review Request: libjpeg-turbo - MMX/SSE accelerated libjpeg
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=600243 --- Comment #25 from Chen Lei 2010-06-14 06:51:39 EDT --- (In reply to comment #24) > (In reply to comment #18) > > Hi Adam, > > Add -static subpackage and more documents to mainpackage and -devel > > subpackage, > > I'll start to review this package. > > As I wrote in the comment #23 I don't think the -static package is needed now. > > I included the "example.c" documentation file in the devel subpackage and > extended description a little, it should be sufficient. > > New spec + SRPM: > http://atkac.fedorapeople.org/libjpeg-turbo/libjpeg-turbo.spec > http://atkac.fedorapeople.org/libjpeg-turbo/libjpeg-turbo-0.0.93-6.fc14.src.rpm > How about to ask this on -devel? I found that many core libraries in fedora provides static libs, e.g. glibc openssl zlib libpng. If we don't provide -static subpackage, we need to obsolete it in the main package. -- 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 600243] Review Request: libjpeg-turbo - MMX/SSE accelerated libjpeg
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=600243 --- Comment #24 from Adam Tkac 2010-06-14 06:39:17 EDT --- (In reply to comment #18) > Hi Adam, > Add -static subpackage and more documents to mainpackage and -devel > subpackage, > I'll start to review this package. As I wrote in the comment #23 I don't think the -static package is needed now. I included the "example.c" documentation file in the devel subpackage and extended description a little, it should be sufficient. New spec + SRPM: http://atkac.fedorapeople.org/libjpeg-turbo/libjpeg-turbo.spec http://atkac.fedorapeople.org/libjpeg-turbo/libjpeg-turbo-0.0.93-6.fc14.src.rpm -- 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 600243] Review Request: libjpeg-turbo - MMX/SSE accelerated libjpeg
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=600243 --- Comment #23 from Adam Tkac 2010-06-14 06:19:37 EDT --- (In reply to comment #22) > Well, nonetheless, you *are* going to get push-back if you remove the static > library. I know because it was tried once already, and not that long ago. > See > bug #186060, bug #215537. Fedora is used for other things than just building > itself. In my opinion it's long time since F7 (libjpeg-turbo will hit F14) so we can "retest" if someone still needs the static libjpeg. I would rather see no static libjpeg.a for now. It might be easily added in the future if someone requests it, libjpeg-turbo project uses libtool so I will drop the "--disable-static" configure flag in the specfile and add -static subpackage, it's pretty simple. If we package -static libjpeg now we won't be able to figure time when we can say "noone uses the static libjpeg.a". -- 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 600243] Review Request: libjpeg-turbo - MMX/SSE accelerated libjpeg
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=600243 --- Comment #22 from Tom Lane 2010-06-14 00:01:00 EDT --- Well, nonetheless, you *are* going to get push-back if you remove the static library. I know because it was tried once already, and not that long ago. See bug #186060, bug #215537. Fedora is used for other things than just building itself. -- 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 600243] Review Request: libjpeg-turbo - MMX/SSE accelerated libjpeg
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=600243 --- Comment #21 from Kalev Lember 2010-06-13 04:33:10 EDT --- (In reply to comment #20) > > --whatrequires libjpeg-static returns no matches, > > Well, of course not. Please review what static linking means. If something in package A links against a static library in package B-static, then A also needs BuildRequires: B-static. Otherwise you just can't build against the static library if it isn't present at build time. My repoquery queried *source* packages for such BuildRequires. Since the result was empty, it is safe to say in current rawhide, no package uses libjpeg-static for building. Is that clearer now, Tom? -- 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 600243] Review Request: libjpeg-turbo - MMX/SSE accelerated libjpeg
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=600243 --- Comment #20 from Tom Lane 2010-06-13 00:42:32 EDT --- > --whatrequires libjpeg-static returns no matches, Well, of course not. Please review what static linking means. -- 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 600243] Review Request: libjpeg-turbo - MMX/SSE accelerated libjpeg
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=600243 Kalev Lember changed: What|Removed |Added CC||ka...@smartlink.ee --- Comment #19 from Kalev Lember 2010-06-12 20:20:10 EDT --- repoquery --enablerepo=rawhide-source --archlist=src --whatrequires libjpeg-static returns no matches, so I'd say no package in Fedora actually uses the -static subpackage. -- 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 600243] Review Request: libjpeg-turbo - MMX/SSE accelerated libjpeg
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=600243 --- Comment #18 from Chen Lei 2010-06-12 05:58:28 EDT --- Hi Adam, Add -static subpackage and more documents to mainpackage and -devel subpackage, I'll start to review this package. See http://cvs.fedoraproject.org/viewvc/devel/libjpeg/libjpeg.spec?revision=1.29&view=markup -- 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 600243] Review Request: libjpeg-turbo - MMX/SSE accelerated libjpeg
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=600243 Chen Lei changed: What|Removed |Added Status|NEW |ASSIGNED AssignedTo|nob...@fedoraproject.org|supercyp...@gmail.com Flag||fedora_requires_release_not ||e? -- 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 600243] Review Request: libjpeg-turbo - MMX/SSE accelerated libjpeg
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=600243 Chen Lei changed: What|Removed |Added Flag|fedora_requires_release_not |fedora-review? |e? | -- 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 600243] Review Request: libjpeg-turbo - MMX/SSE accelerated libjpeg
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=600243 --- Comment #17 from Tom Lane 2010-06-11 11:06:10 EDT --- The static library is used by some boot packages (think boot splash screen). If your ambition is to replace libjpeg 6b altogether, you'd better offer a static library. Obsoleting it without replacing the functionality would certainly be bogus anyway. -- 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 600243] Review Request: libjpeg-turbo - MMX/SSE accelerated libjpeg
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=600243 --- Comment #16 from Chen Lei 2010-06-11 10:55:33 EDT --- The original libjpeg ship static subpackage, is there any package in fedora link with libjpeg-static? Should libjpeg-turbo also need to ship static subpackage or simple obsoletes it? -- 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 600243] Review Request: libjpeg-turbo - MMX/SSE accelerated libjpeg
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=600243 --- Comment #15 from Adam Tkac 2010-06-11 06:14:49 EDT --- (In reply to comment #14) > (In reply to comment #13) > > An intersting question: If we enable SIMD optimazation in libjpeg-turbo, > > can it > > work fine on pure i686 machine without sse2, e.g. OLPC? > > After the debacle with the F-13 GCC breakage, best bet would be to compile > libjpeg-turbo for F-12 (since F-13 does not boot on OLPC right now) and ask > around on their mailing lists for volunteers. > > One would hope that the instructions to choose are dynamically picked at > runtime, but someone familiar with the code should probably answer this. > Adam? It works fine on such machines, I tested it on my Pentium III: [at...@drtic ~]$ cat /proc/cpuinfo |egrep '(model|flags)' model : 8 model name : Pentium III (Coppermine) flags : fpu vme de pse tsc msr pae mce cx8 mtrr pge mca cmov pse36 mmx fxsr sse up libjpeg-turbo code uses the cpuid instruction and dynamically selects the best routines (SSE2/MMX/standard FPU). -- 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 600243] Review Request: libjpeg-turbo - MMX/SSE accelerated libjpeg
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=600243 --- Comment #14 from Michel Alexandre Salim 2010-06-11 04:53:05 EDT --- (In reply to comment #13) > An intersting question: If we enable SIMD optimazation in libjpeg-turbo, can > it > work fine on pure i686 machine without sse2, e.g. OLPC? After the debacle with the F-13 GCC breakage, best bet would be to compile libjpeg-turbo for F-12 (since F-13 does not boot on OLPC right now) and ask around on their mailing lists for volunteers. One would hope that the instructions to choose are dynamically picked at runtime, but someone familiar with the code should probably answer this. Adam? -- 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 600243] Review Request: libjpeg-turbo - MMX/SSE accelerated libjpeg
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=600243 --- Comment #13 from Chen Lei 2010-06-11 04:32:12 EDT --- An intersting question: If we enable SIMD optimazation in libjpeg-turbo, can it work fine on pure i686 machine without sse2, e.g. OLPC? -- 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 600243] Review Request: libjpeg-turbo - MMX/SSE accelerated libjpeg
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=600243 --- Comment #12 from Adam Tkac 2010-06-11 03:58:33 EDT --- (In reply to comment #11) > Do you want the disttag in the obsoletes/provides? I don't think "fc12" looks > right there. Fixed, fetch new spec+SRPM from: http://atkac.fedorapeople.org/libjpeg-turbo/libjpeg-turbo.spec http://atkac.fedorapeople.org/libjpeg-turbo/libjpeg-turbo-0.0.93-5.fc14.src.rpm -- 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 600243] Review Request: libjpeg-turbo - MMX/SSE accelerated libjpeg
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=600243 --- Comment #11 from Adam Goode 2010-06-10 12:53:32 EDT --- Do you want the disttag in the obsoletes/provides? I don't think "fc12" looks right there. -- 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 600243] Review Request: libjpeg-turbo - MMX/SSE accelerated libjpeg
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=600243 Michel Alexandre Salim changed: What|Removed |Added CC||michael.silva...@gmail.com --- Comment #10 from Michel Alexandre Salim 2010-06-10 06:53:57 EDT --- Chen Lei, will you be doing the package review? -- 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 600243] Review Request: libjpeg-turbo - MMX/SSE accelerated libjpeg
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=600243 --- Comment #9 from Adam Tkac 2010-06-10 06:43:42 EDT --- (In reply to comment #8) > Mock building for libjpeg-turbo fails on ppc/ppc64, since libjpeg is a > critical > path package in fedora, I think it should be fixed and build cleanly on most > secondary architectures. Seems I was dazzled by performance improvement on x86 & x64 and I forgot to test SRPM on secondary archs :) Following SRPM should be fine: http://atkac.fedorapeople.org/libjpeg-turbo/libjpeg-turbo.spec http://atkac.fedorapeople.org/libjpeg-turbo/libjpeg-turbo-0.0.93-4.fc14.src.rpm -- 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 600243] Review Request: libjpeg-turbo - MMX/SSE accelerated libjpeg
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=600243 --- Comment #8 from Chen Lei 2010-06-10 05:48:33 EDT --- Mock building for libjpeg-turbo fails on ppc/ppc64, since libjpeg is a critical path package in fedora, I think it should be fixed and build cleanly on most secondary architectures. See http://koji.fedoraproject.org/koji/taskinfo?taskID=2242161 -- 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 600243] Review Request: libjpeg-turbo - MMX/SSE accelerated libjpeg
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=600243 --- Comment #7 from Adam Tkac 2010-06-10 05:02:35 EDT --- (In reply to comment #6) > For packages that are not usually pulled in by using the package name as the > dependency such as library only packages (which are pulled in through library > soname depenencies), there's usually no need to add the Provides. Note however > that the -devel subpackages of lib packages are pulled in as build > dependencies > using the package name, so adding the Provides is often appropriate there. > > So for libjpeg-turbo and libjpeg-turbo-tools provides are not needed. Upgrade > path is not related to provides at all. Your analysis seems valid for me, I modified the obsoletes/provides as you suggested. New spec + SRPM: http://atkac.fedorapeople.org/libjpeg-turbo/libjpeg-turbo.spec http://atkac.fedorapeople.org/libjpeg-turbo/libjpeg-turbo-0.0.93-3.fc14.src.rpm -- 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 600243] Review Request: libjpeg-turbo - MMX/SSE accelerated libjpeg
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=600243 --- Comment #6 from Chen Lei 2010-06-09 10:49:46 EDT --- > > 2. > > Obsoletes: libjpeg < 6b-47.fc12 > > Provides: libjpeg = 6b-47.fc12 > > > > Provides is not needed here. > If I read > http://fedoraproject.org/wiki/PackageNamingGuidelines#Renaming.2Freplacing_existing_packages > correctly it is needed, isn't it? > New spec + SRPM: > http://atkac.fedorapeople.org/libjpeg-turbo/libjpeg-turbo.spec > http://atkac.fedorapeople.org/libjpeg-turbo/libjpeg-turbo-0.0.93-2.fc14.src.rpm > If a package supersedes/replaces an existing package without being a compatible enough replacement as defined in above, use only the Obsoletes from above. For packages that are not usually pulled in by using the package name as the dependency such as library only packages (which are pulled in through library soname depenencies), there's usually no need to add the Provides. Note however that the -devel subpackages of lib packages are pulled in as build dependencies using the package name, so adding the Provides is often appropriate there. So for libjpeg-turbo and libjpeg-turbo-tools provides are not needed. Upgrade path is not related to provides at all. I'm not sure if obsoletes is needed for libjpeg-turbo since the -tools subpackage already depends on soname, you may need check it using yum. yum install libjpeg from official repo. Then koji-build libjpeg-turbo, download them to one directory. Type: createrepo Use a repo file such as this one: [tmp] name=tmp baseurl=file:///tmp/rpm enabled=1 gpgcheck=0 Type: yum upgrade -- 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 600243] Review Request: libjpeg-turbo - MMX/SSE accelerated libjpeg
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=600243 --- Comment #5 from Adam Tkac 2010-06-09 09:03:03 EDT --- (In reply to comment #4) > Wouldn't it be better to improve libjpeg? This question was properly discussed on devel list, check this thread: http://lists.fedoraproject.org/pipermail/devel/2010-May/136556.html Particularly those messages: http://lists.fedoraproject.org/pipermail/devel/2010-June/137052.html http://lists.fedoraproject.org/pipermail/devel/2010-June/137060.html (some parts of libjpeg-turbo are licensed under less restrictive license than libjpeg's license) -- 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 600243] Review Request: libjpeg-turbo - MMX/SSE accelerated libjpeg
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=600243 Paul Wise (Debian) changed: What|Removed |Added CC||pa...@bonedaddy.net --- Comment #4 from Paul Wise (Debian) 2010-06-09 08:44:55 EDT --- Wouldn't it be better to improve libjpeg? -- 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 600243] Review Request: libjpeg-turbo - MMX/SSE accelerated libjpeg
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=600243 --- Comment #3 from Adam Tkac 2010-06-09 08:06:19 EDT --- (In reply to comment #2) > Some trival comment: > 1.Source0: > http://downloads.sourceforge.net/project/libjpeg-turbo/%{version}/libjpeg-turbo-%{version}.tar.gz > > -> > Source0: http://downloads.sourceforge.net/%{name}/%{name}-%{version}.tar.gz > > See http://fedoraproject.org/wiki/Packaging:SourceURL#Sourceforge.net Thanks for the hint, fixed. > 2. > Obsoletes: libjpeg < 6b-47.fc12 > Provides: libjpeg = 6b-47.fc12 > > Provides is not needed here. If I read http://fedoraproject.org/wiki/PackageNamingGuidelines#Renaming.2Freplacing_existing_packages correctly it is needed, isn't it? New spec + SRPM: http://atkac.fedorapeople.org/libjpeg-turbo/libjpeg-turbo.spec http://atkac.fedorapeople.org/libjpeg-turbo/libjpeg-turbo-0.0.93-2.fc14.src.rpm -- 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 600243] Review Request: libjpeg-turbo - MMX/SSE accelerated libjpeg
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=600243 --- Comment #2 from Chen Lei 2010-06-09 03:36:39 EDT --- Some trival comment: 1.Source0: http://downloads.sourceforge.net/project/libjpeg-turbo/%{version}/libjpeg-turbo-%{version}.tar.gz -> Source0: http://downloads.sourceforge.net/%{name}/%{name}-%{version}.tar.gz See http://fedoraproject.org/wiki/Packaging:SourceURL#Sourceforge.net 2. Obsoletes: libjpeg < 6b-47.fc12 Provides: libjpeg = 6b-47.fc12 Provides is not needed here. -- 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 600243] Review Request: libjpeg-turbo - MMX/SSE accelerated libjpeg
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=600243 --- Comment #1 from Adam Tkac 2010-06-04 05:21:01 EDT --- Note for the reviewer - it might look scary that libjpeg-turbo Obsoletes & Provides libjpeg but it is actually a proposed Fedora 14 feature - http://fedoraproject.org/wiki/Features/libjpeg-turbo. -- 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