{"thread":{"id":"13909","subject":"Cleaning up INSTALL before 1.5.6","startedAt":"2008-06-12T07:20:31Z","lastAt":"2008-06-15T13:53:04Z","messageCount":14,"participants":["Johan Herland","David Symonds","Matthieu Moy","Jakub Narebski"],"isPatch":false,"patchVersion":null,"patchTotal":null},"messages":[{"id":"79571","messageId":"200806120920.31161.johan@herland.net","threadId":"13909","inReplyTo":null,"subject":"Cleaning up INSTALL before 1.5.6","fromName":"Johan Herland","fromEmail":"johan@herland.net","sentAt":"2008-06-12T07:20:31Z","receivedAt":"2008-06-12T07:20:31Z","isPatch":false,"sender":{"key":"johan@herland.net","avatar":"https://avatars.githubusercontent.com/u/547031?v=4"},"body":"AFAICS, there's a couple of things that should be changed in the list of \nsystem requirements:\n\n- cpio is no longer needed due to git-clone now being a builtin\n\n- According to some other thread on this list, curl might no longer be \nneeded, as all its users are now builtin, and uses libcurl instead.\n\n\nHave fun! :)\n\n...Johan\n\n-- \nJohan Herland, \nwww.herland.net\n"},{"id":"79914","messageId":"200806151213.22624.johan@herland.net","threadId":"13909","inReplyTo":"200806120920.31161.johan@herland.net","subject":"[PATCH 1/3] cpio is no longer used by git-clone","fromName":"Johan Herland","fromEmail":"johan@herland.net","sentAt":"2008-06-15T10:13:22Z","receivedAt":"2008-06-15T10:13:22Z","isPatch":true,"sender":{"key":"johan@herland.net","avatar":"https://avatars.githubusercontent.com/u/547031?v=4"},"body":"Signed-off-by: Johan Herland \n---\n INSTALL | 3 ---\n 1 files changed, 0 insertions(+), 3 deletions(-)\n\ndiff --git a/INSTALL b/INSTALL\nindex d9b425f..9888f25 100644\n--- a/INSTALL\n+++ b/INSTALL\n@@ -83,9 +83,6 @@ Issues of note:\n \t- \"perl\" and POSIX-compliant shells are needed to use most of\n \t the barebone Porcelainish scripts.\n \n-\t- \"cpio\" is used by git-clone when doing a local (possibly\n-\t hardlinked) clone.\n-\n - Some platform specific issues are dealt with Makefile rules,\n but depending on your specific installation, you may not\n have all the libraries/tools needed, or you may have\n-- \n1.5.6.rc2.128.gf64ae\n"},{"id":"79915","messageId":"200806151214.12525.johan@herland.net","threadId":"13909","inReplyTo":"200806120920.31161.johan@herland.net","subject":"[PATCH 2/3] Consistency: Use \"libcurl\" instead of \"cURL library\" and \"curl\"","fromName":"Johan Herland","fromEmail":"johan@herland.net","sentAt":"2008-06-15T10:14:12Z","receivedAt":"2008-06-15T10:14:12Z","isPatch":true,"sender":{"key":"johan@herland.net","avatar":"https://avatars.githubusercontent.com/u/547031?v=4"},"body":"Signed-off-by: Johan Herland \n---\n Documentation/git-http-push.txt | 4 ++--\n Documentation/howto/setup-git-server-over-http.txt | 2 +-\n Makefile | 2 +-\n configure.ac | 2 +-\n 4 files changed, 5 insertions(+), 5 deletions(-)\n\ndiff --git a/Documentation/git-http-push.txt b/Documentation/git-http-push.txt\nindex 8d32a92..d69b205 100644\n--- a/Documentation/git-http-push.txt\n+++ b/Documentation/git-http-push.txt\n@@ -15,8 +15,8 @@ DESCRIPTION\n Sends missing objects to remote repository, and updates the\n remote branch.\n \n-*NOTE*: This command is temporarily disabled if your cURL\n-library is older than 7.16, as the combination has been reported\n+*NOTE*: This command is temporarily disabled if your libcurl\n+is older than 7.16, as the combination has been reported\n not to work and sometimes corrupts repository.\n \n OPTIONS\ndiff --git a/Documentation/howto/setup-git-server-over-http.txt b/Documentation/howto/setup-git-server-over-http.txt\nindex b7d09c1..4032748 100644\n--- a/Documentation/howto/setup-git-server-over-http.txt\n+++ b/Documentation/howto/setup-git-server-over-http.txt\n@@ -186,7 +186,7 @@ Step 3: setup the client\n ------------------------\n \n Make sure that you have HTTP support, i.e. your git was built with\n-curl (version more recent than 7.10). The command 'git http-push' with\n+libcurl (version more recent than 7.10). The command 'git http-push' with\n no argument should display a usage message.\n \n Then, add the following to your $HOME/.netrc (you can do without, but will be\ndiff --git a/Makefile b/Makefile\nindex 1937507..06ebebe 100644\n--- a/Makefile\n+++ b/Makefile\n@@ -13,7 +13,7 @@ all::\n # Define NO_OPENSSL environment variable if you do not have OpenSSL.\n # This also implies MOZILLA_SHA1.\n #\n-# Define NO_CURL if you do not have curl installed. git-http-pull and\n+# Define NO_CURL if you do not have libcurl installed. git-http-pull and\n # git-http-push are not built, and you cannot use http:// and https://\n # transports.\n #\ndiff --git a/configure.ac b/configure.ac\nindex 82584e9..7c2856e 100644\n--- a/configure.ac\n+++ b/configure.ac\n@@ -158,7 +158,7 @@ AC_CHECK_LIB([crypto], [SHA1_Init],\n AC_SUBST(NEEDS_SSL_WITH_CRYPTO)\n AC_SUBST(NO_OPENSSL)\n #\n-# Define NO_CURL if you do not have curl installed. git-http-pull and\n+# Define NO_CURL if you do not have libcurl installed. git-http-pull and\n # git-http-push are not built, and you cannot use http:// and https://\n # transports.\n AC_CHECK_LIB([curl], [curl_global_init],\n-- \n1.5.6.rc2.128.gf64ae\n"},{"id":"79916","messageId":"200806151216.01886.johan@herland.net","threadId":"13909","inReplyTo":"200806120920.31161.johan@herland.net","subject":"[PATCH 3/3] The \"curl\" executable is no longer required","fromName":"Johan Herland","fromEmail":"johan@herland.net","sentAt":"2008-06-15T10:16:01Z","receivedAt":"2008-06-15T10:16:01Z","isPatch":true,"sender":{"key":"johan@herland.net","avatar":"https://avatars.githubusercontent.com/u/547031?v=4"},"body":"git-clone.sh was the last user of the \"curl\" executable. Relevant git\ncommands now use libcurl instead. This should be reflected in the\ninstall requirements.\n\nSigned-off-by: Johan Herland \n---\n\nI haven't tested the RPM spec change, but it seems fairly straightforward...\n\n INSTALL | 8 ++++----\n git.spec.in | 2 +-\n 2 files changed, 5 insertions(+), 5 deletions(-)\n\ndiff --git a/INSTALL b/INSTALL\nindex 9888f25..4a4e13f 100644\n--- a/INSTALL\n+++ b/INSTALL\n@@ -67,10 +67,10 @@ Issues of note:\n \t that come with git (git includes the one from Mozilla, and has\n \t its own PowerPC and ARM optimized ones too - see the Makefile).\n \n-\t- \"libcurl\" and \"curl\" executable. git-http-fetch and\n-\t git-fetch use them. If you do not use http\n-\t transfer, you are probably OK if you do not have\n-\t them.\n+\t- libcurl library; git-http-fetch and git-fetch use them. You\n+\t might also want the \"curl\" executable for debugging purposes.\n+\t If you do not use http transfer, you are probably OK if you\n+\t do not have them.\n \n \t- expat library; git-http-push uses it for remote lock\n \t management over DAV. Similar to \"curl\" above, this is optional.\ndiff --git a/git.spec.in b/git.spec.in\nindex 97a26be..d946bb1 100644\n--- a/git.spec.in\n+++ b/git.spec.in\n@@ -12,7 +12,7 @@ BuildRequires:\tzlib-devel >= 1.2, openssl-devel, \ncurl-devel, expat-devel, gettex\n BuildRoot:\t%{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)\n \n Requires:\tperl-Git = %{version}-%{release}\n-Requires:\tzlib >= 1.2, rsync, curl, less, openssh-clients, expat\n+Requires:\tzlib >= 1.2, rsync, less, openssh-clients, expat\n Provides:\tgit-core = %{version}-%{release}\n Obsoletes:\tgit-core <= 1.5.4.2\n Obsoletes:\tgit-p4\n-- \n1.5.6.rc2.128.gf64ae\n"},{"id":"79917","messageId":"ee77f5c20806150352n6ccf00e9w8083a17d8c5c2326@mail.gmail.com","threadId":"13909","inReplyTo":"200806151216.01886.johan@herland.net","subject":"Re: [PATCH 3/3] The \"curl\" executable is no longer required","fromName":"David Symonds","fromEmail":"dsymonds@gmail.com","sentAt":"2008-06-15T10:52:43Z","receivedAt":"2008-06-15T10:52:43Z","isPatch":true,"sender":{"key":"dsymonds@gmail.com","avatar":"https://gravatar.com/avatar/b22f5051cbfc11836e36cf7a690e6cde4e225d835e13295ff98d15c7a9ee3c0f?d=mp&s=160"},"body":"On Sun, Jun 15, 2008 at 8:16 PM, Johan Herland wrote:\n\n> diff --git a/git.spec.in b/git.spec.in\n> index 97a26be..d946bb1 100644\n> --- a/git.spec.in\n> +++ b/git.spec.in\n> @@ -12,7 +12,7 @@ BuildRequires: zlib-devel >= 1.2, openssl-devel,\n> curl-devel, expat-devel, gettex\n> BuildRoot: %{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)\n>\n> Requires: perl-Git = %{version}-%{release}\n> -Requires: zlib >= 1.2, rsync, curl, less, openssh-clients, expat\n> +Requires: zlib >= 1.2, rsync, less, openssh-clients, expat\n> Provides: git-core = %{version}-%{release}\n> Obsoletes: git-core <= 1.5.4.2\n> Obsoletes: git-p4\n\nShouldn't libcurl still be listed as a requirement? Or is it statically linked?\n\n\nDave.\n"},{"id":"79918","messageId":"200806151255.41055.johan@herland.net","threadId":"13909","inReplyTo":"ee77f5c20806150352n6ccf00e9w8083a17d8c5c2326@mail.gmail.com","subject":"Re: [PATCH 3/3] The \"curl\" executable is no longer required","fromName":"Johan Herland","fromEmail":"johan@herland.net","sentAt":"2008-06-15T10:55:40Z","receivedAt":"2008-06-15T10:55:40Z","isPatch":true,"sender":{"key":"johan@herland.net","avatar":"https://avatars.githubusercontent.com/u/547031?v=4"},"body":"On Sunday 15 June 2008, David Symonds wrote:\n> On Sun, Jun 15, 2008 at 8:16 PM, Johan Herland wrote:\n> > diff --git a/git.spec.in b/git.spec.in\n> > index 97a26be..d946bb1 100644\n> > --- a/git.spec.in\n> > +++ b/git.spec.in\n> > @@ -12,7 +12,7 @@ BuildRequires: zlib-devel >= 1.2,\n> > openssl-devel, curl-devel, expat-devel, gettex\n> > BuildRoot: \n> > %{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)\n> >\n> > Requires: perl-Git = %{version}-%{release}\n> > -Requires: zlib >= 1.2, rsync, curl, less, openssh-clients, expat\n> > +Requires: zlib >= 1.2, rsync, less, openssh-clients, expat\n> > Provides: git-core = %{version}-%{release}\n> > Obsoletes: git-core <= 1.5.4.2\n> > Obsoletes: git-p4\n>\n> Shouldn't libcurl still be listed as a requirement? Or is it statically\n> linked?\n\n4 lines further up in git.spec.in:\n\nBuildRequires:\tzlib-devel >= 1.2, openssl-devel, curl-devel, expat-devel, \ngettext %{!?_without_docs:, xmlto, asciidoc > 6.0.3}\n\n\nHave fun!\n\n...Johan\n\n-- \nJohan Herland, \nwww.herland.net\n"},{"id":"79919","messageId":"ee77f5c20806150359x2b0e0953l201ead934b3dbecd@mail.gmail.com","threadId":"13909","inReplyTo":"200806151255.41055.johan@herland.net","subject":"Re: [PATCH 3/3] The \"curl\" executable is no longer required","fromName":"David Symonds","fromEmail":"dsymonds@gmail.com","sentAt":"2008-06-15T10:59:08Z","receivedAt":"2008-06-15T10:59:08Z","isPatch":true,"sender":{"key":"dsymonds@gmail.com","avatar":"https://gravatar.com/avatar/b22f5051cbfc11836e36cf7a690e6cde4e225d835e13295ff98d15c7a9ee3c0f?d=mp&s=160"},"body":"On Sun, Jun 15, 2008 at 8:55 PM, Johan Herland wrote:\n\n> On Sunday 15 June 2008, David Symonds wrote:\n\n>> Shouldn't libcurl still be listed as a requirement? Or is it statically\n>> linked?\n>\n> 4 lines further up in git.spec.in:\n>\n> BuildRequires: zlib-devel >= 1.2, openssl-devel, curl-devel, expat-devel,\n> gettext %{!?_without_docs:, xmlto, asciidoc > 6.0.3}\n\nSo does a built RPM inherit that as a requirement? Just curious, since\nI haven't touched RPMs for the better part of a decade.\n\n\nDave.\n"},{"id":"79920","messageId":"vpqmylnkkm4.fsf@bauges.imag.fr","threadId":"13909","inReplyTo":"200806151255.41055.johan@herland.net","subject":"Re: [PATCH 3/3] The \"curl\" executable is no longer required","fromName":"Matthieu Moy","fromEmail":"matthieu.moy@imag.fr","sentAt":"2008-06-15T11:23:15Z","receivedAt":"2008-06-15T11:23:15Z","isPatch":true,"sender":{"key":"git@matthieu-moy.fr","avatar":"https://avatars.githubusercontent.com/u/14709?v=4"},"body":"Johan Herland writes:\n\n> On Sunday 15 June 2008, David Symonds wrote:\n>> On Sun, Jun 15, 2008 at 8:16 PM, Johan Herland wrote:\n>> > diff --git a/git.spec.in b/git.spec.in\n>> > index 97a26be..d946bb1 100644\n>> > --- a/git.spec.in\n>> > +++ b/git.spec.in\n>> > @@ -12,7 +12,7 @@ BuildRequires: zlib-devel >= 1.2,\n>> > openssl-devel, curl-devel, expat-devel, gettex\n>> > BuildRoot: \n>> > %{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)\n>> >\n>> > Requires: perl-Git = %{version}-%{release}\n>> > -Requires: zlib >= 1.2, rsync, curl, less, openssh-clients, expat\n>> > +Requires: zlib >= 1.2, rsync, less, openssh-clients, expat\n>> > Provides: git-core = %{version}-%{release}\n>> > Obsoletes: git-core <= 1.5.4.2\n>> > Obsoletes: git-p4\n>>\n>> Shouldn't libcurl still be listed as a requirement? Or is it statically\n>> linked?\n>\n> 4 lines further up in git.spec.in:\n>\n> BuildRequires:\tzlib-devel >= 1.2, openssl-devel, curl-devel, expat-devel, \n> gettext %{!?_without_docs:, xmlto, asciidoc > 6.0.3}\n\nI don't understand how that answers the question. Unless curl is\nstatically linked, you need the library both at build time and at run\ntime. For example, zlib-devel is needed here at build time, and zlib\n(without devel) is still in \"Requires:\". So, what's the difference\nbetween zlib and libcurl here?\n\n-- \nMatthieu\n"},{"id":"79921","messageId":"200806151340.50694.johan@herland.net","threadId":"13909","inReplyTo":"vpqmylnkkm4.fsf@bauges.imag.fr","subject":"[PATCH 3/3 v2] The \"curl\" executable is no longer required","fromName":"Johan Herland","fromEmail":"johan@herland.net","sentAt":"2008-06-15T11:40:50Z","receivedAt":"2008-06-15T11:40:50Z","isPatch":true,"sender":{"key":"johan@herland.net","avatar":"https://avatars.githubusercontent.com/u/547031?v=4"},"body":"git-clone.sh was the last user of the \"curl\" executable. Relevant git\ncommands now use libcurl instead. This should be reflected in the\ninstall requirements.\n\nSigned-off-by: Johan Herland \n---\n\nOn Sunday 15 June 2008, Matthieu Moy wrote:\n> I don't understand how that answers the question. Unless curl is\n> statically linked, you need the library both at build time and at run\n> time. For example, zlib-devel is needed here at build time, and zlib\n> (without devel) is still in \"Requires:\". So, what's the difference\n> between zlib and libcurl here?\n\nYou're probably right. What about this?\n\n INSTALL | 8 ++++----\n git.spec.in | 2 +-\n 2 files changed, 5 insertions(+), 5 deletions(-)\n\ndiff --git a/INSTALL b/INSTALL\nindex 9888f25..4a4e13f 100644\n--- a/INSTALL\n+++ b/INSTALL\n@@ -67,10 +67,10 @@ Issues of note:\n \t that come with git (git includes the one from Mozilla, and has\n \t its own PowerPC and ARM optimized ones too - see the Makefile).\n \n-\t- \"libcurl\" and \"curl\" executable. git-http-fetch and\n-\t git-fetch use them. If you do not use http\n-\t transfer, you are probably OK if you do not have\n-\t them.\n+\t- libcurl library; git-http-fetch and git-fetch use them. You\n+\t might also want the \"curl\" executable for debugging purposes.\n+\t If you do not use http transfer, you are probably OK if you\n+\t do not have them.\n \n \t- expat library; git-http-push uses it for remote lock\n \t management over DAV. Similar to \"curl\" above, this is optional.\ndiff --git a/git.spec.in b/git.spec.in\nindex 97a26be..fb8da85 100644\n--- a/git.spec.in\n+++ b/git.spec.in\n@@ -12,7 +12,7 @@ BuildRequires:\tzlib-devel >= 1.2, openssl-devel, curl-devel, expat-devel, gettex\n BuildRoot:\t%{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)\n \n Requires:\tperl-Git = %{version}-%{release}\n-Requires:\tzlib >= 1.2, rsync, curl, less, openssh-clients, expat\n+Requires:\tzlib >= 1.2, rsync, libcurl, less, openssh-clients, expat\n Provides:\tgit-core = %{version}-%{release}\n Obsoletes:\tgit-core <= 1.5.4.2\n Obsoletes:\tgit-p4\n-- \n1.5.6.rc2.128.gf64ae\n"},{"id":"79922","messageId":"m3fxreq60t.fsf@localhost.localdomain","threadId":"13909","inReplyTo":"ee77f5c20806150352n6ccf00e9w8083a17d8c5c2326@mail.gmail.com","subject":"Re: [PATCH 3/3] The \"curl\" executable is no longer required","fromName":"Jakub Narebski","fromEmail":"jnareb@gmail.com","sentAt":"2008-06-15T11:42:06Z","receivedAt":"2008-06-15T11:42:06Z","isPatch":true,"sender":{"key":"jnareb@gmail.com","avatar":"https://avatars.githubusercontent.com/u/2706?v=4"},"body":"\"David Symonds\" writes:\n\n> On Sun, Jun 15, 2008 at 8:16 PM, Johan Herland wrote:\n> \n> > diff --git a/git.spec.in b/git.spec.in\n> > index 97a26be..d946bb1 100644\n> > --- a/git.spec.in\n> > +++ b/git.spec.in\n> > @@ -12,7 +12,7 @@ BuildRequires: zlib-devel >= 1.2, openssl-devel,\n> > curl-devel, expat-devel, gettex\n> > BuildRoot: %{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)\n> >\n> > Requires: perl-Git = %{version}-%{release}\n> > -Requires: zlib >= 1.2, rsync, curl, less, openssh-clients, expat\n> > +Requires: zlib >= 1.2, rsync, less, openssh-clients, expat\n> > Provides: git-core = %{version}-%{release}\n> > Obsoletes: git-core <= 1.5.4.2\n> > Obsoletes: git-p4\n> \n> Shouldn't libcurl still be listed as a requirement? Or is it\n> statically linked?\n\nIt is not necessary to enumerate simple library dependencies, as\nrpmbuild _detects_ those dependencies during building binary RPMS.\nOnly unusual requirements (on programs (tools) or on specific version\nof library) are needed to be added in Requires header.\n\nThis has nothing to do with BuildRequires. Besides it worked without\nlibcurl in Requires, so it should continue to work without it,\nisn't it?\n\n-- \nJakub Narebski\nPoland\nShadeHawk on #git\n"},{"id":"79923","messageId":"200806151347.31676.johan@herland.net","threadId":"13909","inReplyTo":"m3fxreq60t.fsf@localhost.localdomain","subject":"Re: [PATCH 3/3] The \"curl\" executable is no longer required","fromName":"Johan Herland","fromEmail":"johan@herland.net","sentAt":"2008-06-15T11:47:31Z","receivedAt":"2008-06-15T11:47:31Z","isPatch":true,"sender":{"key":"johan@herland.net","avatar":"https://avatars.githubusercontent.com/u/547031?v=4"},"body":"On Sunday 15 June 2008, Jakub Narebski wrote:\n> \"David Symonds\" writes:\n> > On Sun, Jun 15, 2008 at 8:16 PM, Johan Herland \nwrote:\n> > > diff --git a/git.spec.in b/git.spec.in\n> > > index 97a26be..d946bb1 100644\n> > > --- a/git.spec.in\n> > > +++ b/git.spec.in\n> > > @@ -12,7 +12,7 @@ BuildRequires: zlib-devel >= 1.2,\n> > > openssl-devel, curl-devel, expat-devel, gettex\n> > > BuildRoot: \n> > > %{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)\n> > >\n> > > Requires: perl-Git = %{version}-%{release}\n> > > -Requires: zlib >= 1.2, rsync, curl, less, openssh-clients,\n> > > expat +Requires: zlib >= 1.2, rsync, less, openssh-clients,\n> > > expat Provides: git-core = %{version}-%{release}\n> > > Obsoletes: git-core <= 1.5.4.2\n> > > Obsoletes: git-p4\n> >\n> > Shouldn't libcurl still be listed as a requirement? Or is it\n> > statically linked?\n>\n> It is not necessary to enumerate simple library dependencies, as\n> rpmbuild _detects_ those dependencies during building binary RPMS.\n> Only unusual requirements (on programs (tools) or on specific version\n> of library) are needed to be added in Requires header.\n>\n> This has nothing to do with BuildRequires. Besides it worked without\n> libcurl in Requires, so it should continue to work without it,\n> isn't it?\n\nUnfortunately I don't know this format at all. There are now two versions of \nthe patch, one without \"Requires: curl\", and one where it's replaced \nby \"Requires: libcurl\". Someone with more RPM knowledge will have to \ndetermine which is right.\n\n...Johan\n\n-- \nJohan Herland, \nwww.herland.net\n"},{"id":"79924","messageId":"vpqej6ylx99.fsf@bauges.imag.fr","threadId":"13909","inReplyTo":"200806151347.31676.johan@herland.net","subject":"Re: [PATCH 3/3] The \"curl\" executable is no longer required","fromName":"Matthieu Moy","fromEmail":"matthieu.moy@imag.fr","sentAt":"2008-06-15T12:04:50Z","receivedAt":"2008-06-15T12:04:50Z","isPatch":true,"sender":{"key":"git@matthieu-moy.fr","avatar":"https://avatars.githubusercontent.com/u/14709?v=4"},"body":"Johan Herland writes:\n\n> On Sunday 15 June 2008, Jakub Narebski wrote:\n>> It is not necessary to enumerate simple library dependencies, as\n>> rpmbuild _detects_ those dependencies during building binary RPMS.\n\nGood to know, and that seems to be a valid argument (\"seems to\"\nbecause of my lack of knowledge of rpm, not lack of trust ;-) ).\n\n>> Besides it worked without libcurl in Requires, so it should\n>> continue to work without it, isn't it?\n\nIt used to have a dependency on curl, which itself requires libcurl,\nso the dependency was implicit.\n\n> Someone with more RPM knowledge will have to determine which is\n> right.\n\n+1\n\n-- \nMatthieu\n"},{"id":"79929","messageId":"200806151533.11742.jnareb@gmail.com","threadId":"13909","inReplyTo":"vpqej6ylx99.fsf@bauges.imag.fr","subject":"Re: [PATCH 3/3] The \"curl\" executable is no longer required","fromName":"Jakub Narebski","fromEmail":"jnareb@gmail.com","sentAt":"2008-06-15T13:33:09Z","receivedAt":"2008-06-15T13:33:09Z","isPatch":true,"sender":{"key":"jnareb@gmail.com","avatar":"https://avatars.githubusercontent.com/u/2706?v=4"},"body":"Matthieu Moy wrote:\n> Johan Herland writes:\n>> On Sunday 15 June 2008, Jakub Narebski wrote:\n>>\n>>> It is not necessary to enumerate simple library dependencies, as\n>>> rpmbuild _detects_ those dependencies during building binary RPMS.\n> \n> Good to know, and that seems to be a valid argument (\"seems to\"\n> because of my lack of knowledge of rpm, not lack of trust ;-) ).\n> \n>>> Besides it worked without libcurl in Requires, so it should\n>>> continue to work without it, isn't it?\n> \n> It used to have a dependency on curl, which itself requires libcurl,\n> so the dependency was implicit.\n> \n>> Someone with more RPM knowledge will have to determine which is\n>> right.\n> \n> +1\n\nOr simply remove 'curl' from 'Requires:' header, and check if rpmbuild\nwould pick up libcurl dependency, and if resulting RPM has libcurl as\nrequirement.\n\nFragment of \"make rpm\" output (word wrapped for better readibility):\n[...]\nProvides: git-core = 1.5.6.rc3.198.g368e0.dirty-1\nRequires(rpmlib): rpmlib(CompressedFileNames) <= 3.0.4-1 \n rpmlib(PartialHardlinkSets) <= 4.0.4-1 \n rpmlib(PayloadFilesHavePrefix) <= 4.0-1 \n rpmlib(VersionedDependencies) <= 3.0.3-1\nRequires: /bin/sh /usr/bin/perl expat less \n libc.so.6 libc.so.6(GLIBC_2.0) libc.so.6(GLIBC_2.1) libc.so.6(GLIBC_2.1.1) libc.so.6(GLIBC_2.1.3) \n libc.so.6(GLIBC_2.2) libc.so.6(GLIBC_2.2.3) libc.so.6(GLIBC_2.3) libc.so.6(GLIBC_2.3.4) \n libcrypto.so.5 libcurl.so.3 libexpat.so.0 libz.so.1\n openssh-clients \n perl >= 0:5.006 perl(Getopt::Long) perl(Git) perl(lib)\n perl(strict) perl(warnings) \n perl-Git = 1.5.6.rc3.198.g368e0.dirty-1\n rsync zlib >= 1.2\nObsoletes: git-core <= 1.5.4.2 git-p4\n[...]\n\n$ rpm -q --requires -p git-1.5.6.rc3.198.g368e0.dirty-1.i386.rpm\n/bin/sh \n/usr/bin/perl \nexpat \nless \nlibc.so.6\n[...]\nlibcrypto.so.5 \nlibcurl.so.3 \nlibexpat.so.0 \nlibz.so.1\n[...]\n\nAs you can see rpmbuild picks up dependency on libcurl.\n\n\n>From what I remember using automatically picked depndencies is\npreferred solution, even if I cannot find it in \"Maximum RPM\" book.\n\nHTH\n-- \nJakub Narebski\nPoland\n"},{"id":"79933","messageId":"200806151553.04563.johan@herland.net","threadId":"13909","inReplyTo":"200806151533.11742.jnareb@gmail.com","subject":"Re: [PATCH 3/3] The \"curl\" executable is no longer required","fromName":"Johan Herland","fromEmail":"johan@herland.net","sentAt":"2008-06-15T13:53:04Z","receivedAt":"2008-06-15T13:53:04Z","isPatch":true,"sender":{"key":"johan@herland.net","avatar":"https://avatars.githubusercontent.com/u/547031?v=4"},"body":"On Sunday 15 June 2008, Jakub Narebski wrote:\n> Or simply remove 'curl' from 'Requires:' header, and check if rpmbuild\n> would pick up libcurl dependency, and if resulting RPM has libcurl as\n> requirement.\n>\n> [...]\n>\n> As you can see rpmbuild picks up dependency on libcurl.\n>\n>\n> From what I remember using automatically picked depndencies is\n> preferred solution, even if I cannot find it in \"Maximum RPM\" book.\n\nSo the conclusion is that v1 of the patch is ok, IIUC.\n\n\nThanks,\n\n...Johan\n\n\n-- \nJohan Herland, \nwww.herland.net\n"}]}