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=550582 Thomas Spura <tomspur(a)fedoraproject.org&gt; changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |tomspur(a)fedoraproject.org --- Comment #2 from Thomas Spura <tomspur(a)fedoraproject.org&gt; 2009-12-26 23:49:07 EDT --- Just a few comments: - https://fedoraproject.org/wiki/Packaging/Guidelines#.25global_preferred_o... - Could you reduce the %files a bit? e.g.: %{_bindir}/dpkg-* %{_mandir}/*/man*/dpkg-* and so on. It will be easier to follow, and maybe the compression format of the man pages changes and you need to rename all *.gz to something else. - There is a new version upstream. How do you create the patch? (Maybe sed would be enought. Most of the changes are only in the *.po file, which will change quite often...) -- 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.

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=550582 --- Comment #4 from leigh scott <leigh123linux(a)googlemail.com&gt; 2009-12-27 06:02:45 EDT --- (In reply to comment #2) The dpkg package is total scrambled if you do this -- 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.

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=550582 --- Comment #6 from Ville Skyttä <ville.skytta(a)iki.fi&gt; 2010-01-03 05:18:20 EDT --- Installing a /usr/sbin/install-info executable seems dangerous as we already have /sbin/install-info. If this executable is required, I suggest installing it somewhere outside of the default $PATH, for example %{_libdir}/dpkg. Development related packages are named *-devel in Fedora instead of *-dev. -- 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.

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=550582 --- Comment #8 from leigh scott <leigh123linux(a)googlemail.com&gt; 2010-01-03 06:48:18 EDT --- (In reply to comment #7) wrong link http://packages.debian.org/sid/all/dpkg-dev/filelist -- 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.

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=550582 Orcan 'oget' Ogetbil <oget.fedora(a)gmail.com&gt; changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED CC| |oget.fedora(a)gmail.com AssignedTo|nobody(a)fedoraproject.org |oget.fedora(a)gmail.com Flag| |fedora-review? --- Comment #10 from Orcan 'oget' Ogetbil <oget.fedora(a)gmail.com&gt; 2010-01-22 17:49:44 EST --- Here is the full review for this: ? rpmlint says: dselect.x86_64: W: only-non-binary-in-usr-lib Do those files have to be in libdir? Please change or defend. * The patch makes references to /usr/lib which is not the right path for x86_64. Why does sed break stuff again? * dev package must be renamed to devel * Please don't duplicate the same doc files in the devel and dselect packages * Please go through the source and make sure you tag all licenses correctly. As far as I can tell most of the source code is GPLv2+. However there are some exceptions, such as lib/dpkg/md5.c is public domain lib/dpkg/showpkg.c is GPLv2 etc These have to be listed as comments in the SPEC file. * Source0 seems to be wrong. Gives 404 to me. * Please update to the latest upstream ? Shouldn't the dselect package require the main package? ? Why are we removing the alternatives stuff? * A few directories are owned by both the main package and the devel package. Since the devel package requires the main package, please get rid of the multiple ownership. Also one directory is also owned by the dselect package. If dselect package does not need to require the main package, this is OK. * /usr/sbin/install-info seems dangerous. Please move somewhere else. - At a few places in the build I see the -O1 flag overriding our default flag. This -O1 is passed during the linking phase only. I don't think this is a problem. It is weird though. * I can't install the devel package: $ sudo rpm -ivh dpkg-dev-1.15.4.1-1.fc12.x86_64.rpm error: Failed dependencies: perl(extra) is needed by dpkg-dev-1.15.4.1-1.fc12.x86_64 $ sudo yum install "perl(extra)" ... No package perl(extra) available. -- 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.

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=550582 --- Comment #12 from Andrew Colin Kissa <andrew(a)topdog.za.net&gt; 2010-01-24 02:29:18 EST --- Thanks Oget, I will make the requested changes sometime this week. -- 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.

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=550582 --- Comment #14 from Matt Taggart <taggart(a)debian.org&gt; 2010-01-28 16:40:34 EST --- I was talking to one of the dpkg developers on IRC. He doesn't have a bugzilla account and so asked that I forward these comments on. <guillem> re dpkg in fedora I don't have an account on their bugzilla but you might wanna comment to maybe speed up the review <guillem> install-info is not really needed at all if they are using the one from GNU texinfo <guillem> update-alternatives I thought they have their own implentation (but I might be wrong) <guillem> I see in their spec file they are working around a problem now fixed in the latest released version about selinux linking <guillem> the LDFLAGS='-lselinux' should be unnecessary <guillem> I don't know how they requires stuff gets generated but I suspect the perl requires involving dpkg-gettext.pl and controllib.pl is not needed anymore <guillem> re licensing I did an audit that might be useful to them: http://lists.debian.org/debian-dpkg/2009/11/msg00043.html <guillem> cpio should not be needed by the devel package <guillem> I've not checked the patch they are applying but I'd gladly fix anything that causes them to need to patch it <guillem> there's also been reports on the list about it building fine with recentish git on fedora <guillem> I think that's mostly it <guillem> and thanks for fwding the info :) <guillem> oh and re the linker optimization they can use --disable-linker-optimisations <guillem> and dselect needs dpkg yes <guillem> anyway dinner time for me, poke if you need more info! -- 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.

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=550582 --- Comment #16 from Orcan 'oget' Ogetbil <oget.fedora(a)gmail.com&gt; 2010-02-13 13:47:04 EST --- Thanks, we are almost there. I have a few questions, comments and a few minor issues: ? The only rpmlint is: dpkg.src:1: E: hardcoded-library-path in /usr/lib which is ironically a lot better than our own rpm package. Anyway, rpmlint is an rpmlint. I fail to understand that why we hardcode the libdir to /usr/lib. I read your comment, but is it not possible to patch those perl scripts to look at /usr/share instead of /usr/lib? On the other hand our rpm package also puts files in /usr/lib. I wonder if this is really a must. ! The devel package looks like it can be made noarch. I don't know if this would screw up our build system though. (Meanwhile dpkg and dselect packages cannot be noarch) * The following directories and possibly their subdirectories are not owned: /usr/lib/perl5/vendor_perl/5.10.0/Debian/ %dpkgdir/dpkg/methods/ * Build process log does not show us what flags are used during compilation. Please make it verbose. -- 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.

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=550582 --- Comment #18 from Andrew Colin Kissa <andrew(a)topdog.za.net&gt; 2010-02-13 13:57:17 EST --- Actually i put the files in /usr/lib based on our own rpm package, let me see if i can patch it to move the files into /usr/share. Will update and post. -- 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.

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=550582 --- Comment #20 from Orcan 'oget' Ogetbil <oget.fedora(a)gmail.com&gt; 2010-02-14 15:05:18 EST --- Thanks. (In reply to comment #19) Still there are issues. Please install all the generated packages. Then uninstall them. You will be left with these empty directories /usr/lib/perl5/vendor_perl/5.10.0/Dpkg/ /usr/lib/perl5/vendor_perl/5.10.0/Debian/ /usr/share/dpkg/ That is because their subdirectories are not owned. And rpm does not remove non-empty directories during uninstallation. An example of a way to fix this is to change the lines %dir %{perl_vendorlib}/Debian %{perl_vendorlib}/Debian/Dselect/*.pm to %{perl_vendorlib}/Debian/ then you won't have this problem. Similarly for the other directories. Beware though, %{_datadir}/dpkg/ is used by both the main package and the dselect package so it needs to be handled carefully. Also, you have these two lines under %files: %dir %{_datarootdir}/dpkg %dir %{_datadir}/dpkg As far as I know, %{_datarootdir} and %{_datadir} both point to /usr/share. There is no need to list the same directory twice in the %files listings. I prefer using either one notation or the other. Just pass --disable-silent-rules to configure. ! One last thing: Changing install -m0644 to install -pm0644 will preserve timestamps. No big deal though. -- 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.

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=550582 Orcan 'oget' Ogetbil <oget.fedora(a)gmail.com&gt; changed: What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-review? |fedora-review+ --- Comment #22 from Orcan 'oget' Ogetbil <oget.fedora(a)gmail.com&gt; 2010-02-15 23:03:16 EST --- ... and we are done! --------------------------------------- This package (dpkg) is APPROVED by oget --------------------------------------- Thank you Andrew for packaging this. -- 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.

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=550582 --- Comment #24 from Andrew Colin Kissa <andrew(a)topdog.za.net&gt; 2010-02-17 08:21:53 EST --- Since branching has already taken place. requesting fedora 13 as well. New Package CVS Request ======================= Package Name: dpkg Short Description: Package maintenance system for Debian Linux Owners: topdog Branches: F-11 F-12 F-13 EL-5 -- 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.

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=550582 --- Comment #26 from Fedora Update System <updates(a)fedoraproject.org&gt; 2010-02-20 03:36:57 EST --- dpkg-1.15.5.6-3.fc12 has been submitted as an update for Fedora 12. http://admin.fedoraproject.org/updates/dpkg-1.15.5.6-3.fc12 -- 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.

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=550582 --- Comment #28 from Fedora Update System <updates(a)fedoraproject.org&gt; 2010-02-20 03:37:08 EST --- dpkg-1.15.5.6-3.fc13 has been submitted as an update for Fedora 13. http://admin.fedoraproject.org/updates/dpkg-1.15.5.6-3.fc13 -- 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.

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=550582 --- Comment #30 from Fedora Update System <updates(a)fedoraproject.org&gt; 2010-02-20 17:51:03 EST --- dpkg-1.15.5.6-3.fc12 has been pushed to the Fedora 12 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing update dpkg'. You can provide feedback for this update here: http://admin.fedoraproject.org/updates/F12/FEDORA-2010-1740 -- 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.

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=550582 Fedora Update System <updates(a)fedoraproject.org&gt; changed: What |Removed |Added ---------------------------------------------------------------------------- Status|ON_QA |CLOSED Fixed In Version| |dpkg-1.15.5.6-3.fc12 Resolution| |ERRATA -- 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.

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=550582 Fedora Update System <updates(a)fedoraproject.org&gt; changed: What |Removed |Added ---------------------------------------------------------------------------- Fixed In Version|dpkg-1.15.5.6-3.fc12 |dpkg-1.15.5.6-3.fc13 -- 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.

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=550582 --- Comment #34 from Fedora Update System <updates(a)fedoraproject.org&gt; 2010-03-08 22:37:42 EST --- dpkg-1.15.5.6-3.fc11 has been pushed to the Fedora 11 stable repository. If problems still persist, please make note of it in this bug report. -- 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.

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=550582 --- Comment #35 from Fedora Update System <updates(a)fedoraproject.org&gt; 2011-02-10 13:52:07 EST --- dpkg-1.15.5.6-6.el5 has been submitted as an update for Fedora EPEL 5. https://admin.fedoraproject.org/updates/dpkg-1.15.5.6-6.el5 -- 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.

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=550582 Fedora Update System <updates(a)fedoraproject.org&gt; changed: What |Removed |Added ---------------------------------------------------------------------------- Fixed In Version|dpkg-1.15.5.6-3.fc11 |dpkg-1.15.5.6-6.el5 -- 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.