{"thread":{"id":"1317","subject":"[PATCH] Deb Packaging fixes: Build against Mozilla libs for Debian, conflict with \"git\"","startedAt":"2005-07-23T07:37:07Z","lastAt":"2005-07-25T04:27:45Z","messageCount":9,"participants":["Ryan Anderson","Junio C Hamano","Linus Torvalds","Alexey Nezhdanov"],"isPatch":true,"patchVersion":1,"patchTotal":null},"messages":[{"id":"6369","messageId":"20050723073707.GA3255@mythryan2.michonline.com","threadId":"1317","inReplyTo":null,"subject":"[PATCH] Deb Packaging fixes: Build against Mozilla libs for Debian, conflict with \"git\"","fromName":"Ryan Anderson","fromEmail":"ryan@michonline.com","sentAt":"2005-07-23T07:37:07Z","receivedAt":"2005-07-23T07:37:07Z","isPatch":true,"sender":{"key":"ryan@michonline.com","avatar":null},"body":"\nThis patch includes two fixes to the git-core Debian package:\n\n    * Conflict with the GNU Interactive Tools package, which _also_\n      wants to install /usr/bin/git.\n\n    * Compile against the unencumbered Mozilla SHA1 code, instead of\n      the iffy OpenSSL code.  This makes it easier to get the package\n      included for distribution with Debian.\n\nNote: Assumes that Ryan Anderson's patch \"Deb packages should include\nthe binaries\" has been applied.\n\n(Note - I fixed up the conflicts against my earlier packaging fixes -- Ryan)\n\nSigned-off-by: Sebastian Kuzminsky <seb@highlab.com>\nSigned-off-by: Ryan Anderson <ryan@michonline.com>\n---\n\n debian/changelog |   11 +++++++++++\n debian/control   |    3 ++-\n debian/rules     |   14 ++++++++++++++\n 3 files changed, 27 insertions(+), 1 deletions(-)\n\nc651cdc175712c26d4b9c8b75b44a89eb1b74752\ndiff --git a/debian/changelog b/debian/changelog\n--- a/debian/changelog\n+++ b/debian/changelog\n@@ -1,3 +1,14 @@\n+git-core (0.99.1-1) unstable; urgency=low\n+\n+  * Conflict with the GNU Interactive Tools package, which also installs\n+    /usr/bin/git.\n+  * Dont compile against the OpenSSL SHA1 code, it's problematically\n+    licensed.  Instead use the PPC assembly on PPC, and the code ripped\n+    from Mozilla everywhere else.\n+  * Minor tweaks to the Build-Depends.\n+\n+ -- Sebastian Kuzminsky <seb@highlab.com>  Thu, 21 Jul 2005 01:28:35 -0600\n+\n git-core (0.99-1) unstable; urgency=low\n \n   * Update deb package support to build correctly. \ndiff --git a/debian/control b/debian/control\n--- a/debian/control\n+++ b/debian/control\n@@ -7,7 +7,8 @@ Standards-Version: 3.6.1\n \n Package: git-core\n Architecture: any\n-Depends: ${misc:Depends}, shellutils, diff, rsync, rcs\n+Depends: ${misc:Depends}, patch, diff, rsync, rcs, wget, rsh-client\n+Conflicts: git\n Description: The git content addressable filesystem\n  GIT comes in two layers. The bottom layer is merely an extremely fast\n  and flexible filesystem-based database designed to store directory trees\ndiff --git a/debian/rules b/debian/rules\n--- a/debian/rules\n+++ b/debian/rules\n@@ -12,6 +12,20 @@ else\n endif\n export CFLAGS\n \n+#\n+# On PowerPC we compile against the hand-crafted assembly, on all\n+# other architectures we compile against GPL'ed sha1 code lifted\n+# from Mozilla.  OpenSSL is strangely licensed and best avoided\n+# in Debian.\n+#\n+HOST_ARCH=$(shell dpkg-architecture -qDEB_HOST_ARCH)\n+ifeq (${HOST_ARCH},powerpc)\n+\texport PPC_SHA1=YesPlease\n+else\n+\texport MOZILLA_SHA1=YesPlease\n+endif\n+\n+\n PREFIX := /usr\n MANDIR := /usr/share/man/\n \n-- \n\nRyan Anderson\n  sometimes Pug Majere\n"},{"id":"6377","messageId":"7vd5p9horr.fsf@totally-fudged-out-message-id","threadId":"1317","inReplyTo":"20050723073707.GA3255@mythryan2.michonline.com","subject":"Re: [PATCH] Deb Packaging fixes: Build against Mozilla libs for Debian, conflict with \"git\"","fromName":"Junio C Hamano","fromEmail":"junio@twinsun.com","sentAt":"2005-07-23T09:11:16Z","receivedAt":"2005-07-23T09:11:16Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Ryan Anderson <ryan@michonline.com> writes:\n\n> --- a/debian/changelog\n> +++ b/debian/changelog\n> ...\n> +  * Minor tweaks to the Build-Depends.\n\nThis is a nit and not the reason for NACK, but I do not see any\nchange to Build-Depends.\n\n> -Depends: ${misc:Depends}, shellutils, diff, rsync, rcs\n> +Depends: ${misc:Depends}, patch, diff, rsync, rcs, wget, rsh-client\n\nThis is primarily my fault, but this new Depends line is already\nobsolete.  Darrin Thompson removed the last remaining use of\nwget and it is my understanding that we do not depend on wget\nanymore; instead we now depend on curl executable.\n\nI do not offhand remember where we use rsh-client.  The\nrsh-client I know of is this one, which claims to offer rsh, rcp\nand rlogin but I do not think we use any of them.  Did you mean\n\"ssh\" package?\n\n    Package: rsh-client\n    Priority: extra\n    Section: net\n    Maintainer: Alberto Gonzalez Iniesta <agi@inittab.org>\n    Source: netkit-rsh\n    Version: 0.17-13\n    Description: rsh clients.\n     This package contains rsh, rcp and rlogin.\n\nPerhaps (I am not sure about the rsh-client vs ssh):\n\n> +Depends: ${misc:Depends}, patch, diff, rsync, rcs, curl, ssh\n\nBoth the use of mozilla SHA1 library and conflicting with the\nother GIT, which are the primary points of this patch, sound\nsensible, relative to the Debian poli(cies|tics), but as long as\nwe are touching the Depends: line, let's make sure we get it\nright (the current one is obviously obsolete).\n"},{"id":"6384","messageId":"20050723143936.GE3255@mythryan2.michonline.com","threadId":"1317","inReplyTo":"7vd5p9horr.fsf@totally-fudged-out-message-id","subject":"Re: [PATCH] Deb Packaging fixes: Build against Mozilla libs for Debian, conflict with \"git\"","fromName":"Ryan Anderson","fromEmail":"ryan@michonline.com","sentAt":"2005-07-23T14:39:36Z","receivedAt":"2005-07-23T14:39:36Z","isPatch":true,"sender":{"key":"ryan@michonline.com","avatar":null},"body":"On Sat, Jul 23, 2005 at 02:11:16AM -0700, Junio C Hamano wrote:\n> Ryan Anderson <ryan@michonline.com> writes:\n> \n> > --- a/debian/changelog\n> > +++ b/debian/changelog\n> > ...\n> > +  * Minor tweaks to the Build-Depends.\n> \n> This is a nit and not the reason for NACK, but I do not see any\n> change to Build-Depends.\n> \n> > -Depends: ${misc:Depends}, shellutils, diff, rsync, rcs\n> > +Depends: ${misc:Depends}, patch, diff, rsync, rcs, wget, rsh-client\n\nJust to be clear - I was forwarding Sebastian's patch on, after fixing\nup the conflict I got applying it, so I didn't edit the comments, other\nthan to add my Signed-off-by line.\n \n> This is primarily my fault, but this new Depends line is already\n> obsolete.  Darrin Thompson removed the last remaining use of\n> wget and it is my understanding that we do not depend on wget\n> anymore; instead we now depend on curl executable.\n> \n> I do not offhand remember where we use rsh-client.  The\n> rsh-client I know of is this one, which claims to offer rsh, rcp\n> and rlogin but I do not think we use any of them.  Did you mean\n> \"ssh\" package?\n\nssh Provides rsh-client.\n\nI suspect that was the thinking.  I think the way we use ssh should be\ncompatible with rsh, and that was the underlying reason for doing it\nthis way.\n\n> Both the use of mozilla SHA1 library and conflicting with the\n> other GIT, which are the primary points of this patch, sound\n> sensible, relative to the Debian poli(cies|tics), but as long as\n> we are touching the Depends: line, let's make sure we get it\n> right (the current one is obviously obsolete).\n\nI have no issue with the hunk being dropped or just doing a followup to\nfix things up.  I'll wait until Linus settles back in and our massive\nbacklog is fully committed or commented on.\n\n-- \n\nRyan Anderson\n  sometimes Pug Majere\n"},{"id":"6386","messageId":"Pine.LNX.4.58.0507230921320.6074@g5.osdl.org","threadId":"1317","inReplyTo":"20050723073707.GA3255@mythryan2.michonline.com","subject":"Re: [PATCH] Deb Packaging fixes: Build against Mozilla libs for Debian, conflict with \"git\"","fromName":"Linus Torvalds","fromEmail":"torvalds@osdl.org","sentAt":"2005-07-23T16:24:33Z","receivedAt":"2005-07-23T16:24:33Z","isPatch":true,"sender":{"key":"torvalds@linux-foundation.org","avatar":"https://avatars.githubusercontent.com/u/1024025?v=4"},"body":"\n\nOn Sat, 23 Jul 2005, Ryan Anderson wrote:\n> \n>     * Compile against the unencumbered Mozilla SHA1 code, instead of\n>       the iffy OpenSSL code.  This makes it easier to get the package\n>       included for distribution with Debian.\n\nNote that this is just not true.\n\nWe still use openssl for the bignum stuff in epoch.c, so using the mozilla\nSHA1 libraries just doesn't make any difference at all from an openssl \nstandpoint. You'd have to disable the \"--merge-order\" flag entirely to get \nrid of the openssl dependency, methinks.\n\n\t\tLinus\n"},{"id":"6391","messageId":"20050723192335.GA24071@mythryan2.michonline.com","threadId":"1317","inReplyTo":"Pine.LNX.4.58.0507230921320.6074@g5.osdl.org","subject":"Re: [PATCH] Deb Packaging fixes: Build against Mozilla libs for Debian, conflict with \"git\"","fromName":"Ryan Anderson","fromEmail":"ryan@michonline.com","sentAt":"2005-07-23T19:23:35Z","receivedAt":"2005-07-23T19:23:35Z","isPatch":true,"sender":{"key":"ryan@michonline.com","avatar":null},"body":"On Sat, Jul 23, 2005 at 09:24:33AM -0700, Linus Torvalds wrote:\n> \n> \n> On Sat, 23 Jul 2005, Ryan Anderson wrote:\n> > \n> >     * Compile against the unencumbered Mozilla SHA1 code, instead of\n> >       the iffy OpenSSL code.  This makes it easier to get the package\n> >       included for distribution with Debian.\n> \n> Note that this is just not true.\n> \n> We still use openssl for the bignum stuff in epoch.c, so using the mozilla\n> SHA1 libraries just doesn't make any difference at all from an openssl \n> standpoint. You'd have to disable the \"--merge-order\" flag entirely to get \n> rid of the openssl dependency, methinks.\n\nI think that's going a bit far for the purpose of this packaging.\n\nI reworked the comments in the changelog to make it clear that the\nelimination of OpenSSL is not complete, and also address the comments\nJunio made.\n\nI'll reply to this email with that updated patch, but I've got this and\nall the other pending Debian packaging fixes in\nrsync://h4x0r5.com/git-ryan.git/\n\n-- \n\nRyan Anderson\n  sometimes Pug Majere\n"},{"id":"6392","messageId":"20050723192632.GB24071@mythryan2.michonline.com","threadId":"1317","inReplyTo":"20050723192335.GA24071@mythryan2.michonline.com","subject":"[PATCH] Deb Packaging fixes: Build against Mozilla libs for Debian, conflict with \"git\"","fromName":"Ryan Anderson","fromEmail":"ryan@michonline.com","sentAt":"2005-07-23T19:26:32Z","receivedAt":"2005-07-23T19:26:32Z","isPatch":true,"sender":{"key":"ryan@michonline.com","avatar":null},"body":"This patch includes two fixes to the git-core Debian package:\n\n    * Conflict with the GNU Interactive Tools package, which _also_\n      wants to install /usr/bin/git.\n\n    * Compile against the unencumbered Mozilla SHA1 code, instead of\n      the iffy OpenSSL code, as much as possible.  This makes it easier to get\n      the package included for distribution with Debian.\n\nThis has been based upon the original patch by Sebastian Kuzminsky\n<seb@highlab.com>, but has been fixed up based upon feedback.\n\nSigned-off-by: Ryan Anderson <ryan@michonline.com>\n---\n\n changelog |   10 ++++++++++\n control   |    3 ++-\n rules     |   14 ++++++++++++++\n 3 files changed, 26 insertions(+), 1 deletion(-)\n\ndiff --git a/debian/changelog b/debian/changelog\n--- a/debian/changelog\n+++ b/debian/changelog\n@@ -1,3 +1,13 @@\n+git-core (0.99-2) unstable; urgency=low\n+\n+  * Conflict with the GNU Interactive Tools package, which also installs\n+    /usr/bin/git.\n+  * Use the Mozilla SHA1 code and/or the PPC assembly in preference to\n+    OpenSSL.  This is only a partial fix for the license issues with OpenSSL.\n+  * Minor tweaks to the Depends.\n+\n+ -- Ryan Anderson <ryan@michonline.com>  Sat, 23 Jul 2005 14:15:00 -0400\n+\n git-core (0.99-1) unstable; urgency=low\n \n   * Update deb package support to build correctly. \ndiff --git a/debian/control b/debian/control\n--- a/debian/control\n+++ b/debian/control\n@@ -7,7 +7,8 @@ Standards-Version: 3.6.1\n \n Package: git-core\n Architecture: any\n-Depends: ${misc:Depends}, shellutils, diff, rsync, rcs\n+Depends: ${misc:Depends}, patch, diff, rsync, rcs, ssh\n+Conflicts: git\n Description: The git content addressable filesystem\n  GIT comes in two layers. The bottom layer is merely an extremely fast\n  and flexible filesystem-based database designed to store directory trees\ndiff --git a/debian/rules b/debian/rules\n--- a/debian/rules\n+++ b/debian/rules\n@@ -12,6 +12,20 @@ else\n endif\n export CFLAGS\n \n+#\n+# On PowerPC we compile against the hand-crafted assembly, on all\n+# other architectures we compile against GPL'ed sha1 code lifted\n+# from Mozilla.  OpenSSL is strangely licensed and best avoided\n+# in Debian.\n+#\n+HOST_ARCH=$(shell dpkg-architecture -qDEB_HOST_ARCH)\n+ifeq (${HOST_ARCH},powerpc)\n+\texport PPC_SHA1=YesPlease\n+else\n+\texport MOZILLA_SHA1=YesPlease\n+endif\n+\n+\n PREFIX := /usr\n MANDIR := /usr/share/man/\n \n\n-- \n\nRyan Anderson\n  sometimes Pug Majere\n"},{"id":"6405","messageId":"200507240728.40158.snake@penza-gsm.ru","threadId":"1317","inReplyTo":"20050723192632.GB24071@mythryan2.michonline.com","subject":"Re: [PATCH] Deb Packaging fixes: Build against Mozilla libs for Debian, conflict with \"git\"","fromName":"Alexey Nezhdanov","fromEmail":"snake@penza-gsm.ru","sentAt":"2005-07-24T03:28:40Z","receivedAt":"2005-07-24T03:28:40Z","isPatch":true,"sender":{"key":"snake@penza-gsm.ru","avatar":null},"body":"Satturday, 23 July 2005 23:26 Ryan Anderson wrote:\n> -Depends: ${misc:Depends}, shellutils, diff, rsync, rcs\n> +Depends: ${misc:Depends}, patch, diff, rsync, rcs, ssh\nDid I missed something or you forgot about libcurl3 dependency?\n\n-- \nRespectfully\nAlexey Nezhdanov\n"},{"id":"6406","messageId":"7vwtngg2bd.fsf@assigned-by-dhcp.cox.net","threadId":"1317","inReplyTo":"200507240728.40158.snake@penza-gsm.ru","subject":"Re: [PATCH] Deb Packaging fixes: Build against Mozilla libs for Debian, conflict with \"git\"","fromName":"Junio C Hamano","fromEmail":"junkio@cox.net","sentAt":"2005-07-24T06:12:22Z","receivedAt":"2005-07-24T06:12:22Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Alexey Nezhdanov <snake@penza-gsm.ru> writes:\n\n> Satturday, 23 July 2005 23:26 Ryan Anderson wrote:\n>> -Depends: ${misc:Depends}, shellutils, diff, rsync, rcs\n>> +Depends: ${misc:Depends}, patch, diff, rsync, rcs, ssh\n> Did I missed something or you forgot about libcurl3 dependency?\n\nI think you are right.  In the build process, dh_shlibdeps is\nused and shlib:Depends is created to include the libcurl3 (among\nother things) in debian/git-core.substvars, but it is not\nactually used in the resulting binary package because the line\nmisses ${shlibs:Depends} there.\n\nRyan, would this change be enough?  I do not know what I am\ndoing (${misc:Depends} is new to me), but this patch seems to\nfix it on my box.\n\n------------\nMake sure binary debian package depends on shlibs it uses.\n\nThe \"Depends:\" line in debian/control lacked ${shlibs:Depends},\nwhich caused the resulting binary package not to depend on\nlibcurl3 nor even libc6 ;-).\n\nSigned-off-by: Junio C Hamano <junkio@cox.net>\n---\n\n# - linus: Fix up applymbox script for the addition of \"git-\" prefix\n# + (working tree)\ndiff --git a/debian/control b/debian/control\n--- a/debian/control\n+++ b/debian/control\n@@ -7,7 +7,7 @@ Standards-Version: 3.6.1\n \n Package: git-core\n Architecture: any\n-Depends: ${misc:Depends}, shellutils, diff, rsync, rcs\n+Depends: ${shlibs:Depends} ${misc:Depends}, shellutils, diff, rsync, rcs\n Description: The git content addressable filesystem\n  GIT comes in two layers. The bottom layer is merely an extremely fast\n  and flexible filesystem-based database designed to store directory trees\n"},{"id":"6418","messageId":"20050725042745.GA6098@mythryan2.michonline.com","threadId":"1317","inReplyTo":"7vwtngg2bd.fsf@assigned-by-dhcp.cox.net","subject":"Re: [PATCH] Deb Packaging fixes: Build against Mozilla libs for Debian, conflict with \"git\"","fromName":"Ryan Anderson","fromEmail":"ryan@michonline.com","sentAt":"2005-07-25T04:27:45Z","receivedAt":"2005-07-25T04:27:45Z","isPatch":true,"sender":{"key":"ryan@michonline.com","avatar":null},"body":"On Sat, Jul 23, 2005 at 11:12:22PM -0700, Junio C Hamano wrote:\n> Alexey Nezhdanov <snake@penza-gsm.ru> writes:\n> \n> > Satturday, 23 July 2005 23:26 Ryan Anderson wrote:\n> >> -Depends: ${misc:Depends}, shellutils, diff, rsync, rcs\n> >> +Depends: ${misc:Depends}, patch, diff, rsync, rcs, ssh\n> > Did I missed something or you forgot about libcurl3 dependency?\n> \n> I think you are right.  In the build process, dh_shlibdeps is\n> used and shlib:Depends is created to include the libcurl3 (among\n> other things) in debian/git-core.substvars, but it is not\n> actually used in the resulting binary package because the line\n> misses ${shlibs:Depends} there.\n> \n> Ryan, would this change be enough?  I do not know what I am\n> doing (${misc:Depends} is new to me), but this patch seems to\n> fix it on my box.\n\nYes - this looks like the right idea - but you're missing a comma\nbetween the two variables.  (That will cause the *entire* depends line\nto get dropped, btw.)\n\nI changed it from shlibs:Depends to misc:Depends because I saw something\nwhen I was tracking down exactly what I needed to put into debian/compat\n(in fact, man debhelper, then search for \"compat\" and look under V4),\nbut I misread \"supplements\" to be something else. \n\nSo, Acked-By: me, if you put the comma in. :)\n\n\n\n> ------------\n> Make sure binary debian package depends on shlibs it uses.\n> \n> The \"Depends:\" line in debian/control lacked ${shlibs:Depends},\n> which caused the resulting binary package not to depend on\n> libcurl3 nor even libc6 ;-).\n> \n> Signed-off-by: Junio C Hamano <junkio@cox.net>\n> ---\n> \n> # - linus: Fix up applymbox script for the addition of \"git-\" prefix\n> # + (working tree)\n> diff --git a/debian/control b/debian/control\n> --- a/debian/control\n> +++ b/debian/control\n> @@ -7,7 +7,7 @@ Standards-Version: 3.6.1\n>  \n>  Package: git-core\n>  Architecture: any\n> -Depends: ${misc:Depends}, shellutils, diff, rsync, rcs\n> +Depends: ${shlibs:Depends} ${misc:Depends}, shellutils, diff, rsync, rcs\n>  Description: The git content addressable filesystem\n>   GIT comes in two layers. The bottom layer is merely an extremely fast\n>   and flexible filesystem-based database designed to store directory trees\n> \n> \n> \n> -\n> To unsubscribe from this list: send the line \"unsubscribe git\" in\n> the body of a message to majordomo@vger.kernel.org\n> More majordomo info at  http://vger.kernel.org/majordomo-info.html\n> \n\n-- \n\nRyan Anderson\n  sometimes Pug Majere\n"}]}