{"thread":{"id":"11499","subject":"[PATCH] Make the git metapackage require the same version of the subpackages.","startedAt":"2008-01-06T17:35:02Z","lastAt":"2008-01-06T21:24:30Z","messageCount":4,"participants":["James Bowes","Junio C Hamano"],"isPatch":true,"patchVersion":1,"patchTotal":null},"messages":[{"id":"64597","messageId":"20080106173501.GB9349@spitfire","threadId":"11499","inReplyTo":null,"subject":"[PATCH] Make the git metapackage require the same version of the subpackages.","fromName":"James Bowes","fromEmail":"jbowes@dangerouslyinc.com","sentAt":"2008-01-06T17:35:02Z","receivedAt":"2008-01-06T17:35:02Z","isPatch":true,"sender":{"key":"jbowes@dangerouslyinc.com","avatar":"https://gravatar.com/avatar/a2fe98c66b2b47a9fa9d2ba92ff949d54c3208b1f8acc2e745b4b84ae3c4483a?d=mp&s=160"},"body":"Without explicit version deps in the rpm spec file, 'yum update git'\neffectively does nothing. Require explicit versions of the subpackages, so that\nthey get pulled in on an update.\n\nSigned-off-by: James Bowes \n---\n git.spec.in | 13 ++++++++++++-\n 1 files changed, 12 insertions(+), 1 deletions(-)\n\ndiff --git a/git.spec.in b/git.spec.in\nindex 3e5bebb..7f1bd5a 100644\n--- a/git.spec.in\n+++ b/git.spec.in\n@@ -10,7 +10,15 @@ URL: \t\thttp://kernel.org/pub/software/scm/git/\n Source: \thttp://kernel.org/pub/software/scm/git/%{name}-%{version}.tar.gz\n BuildRequires:\tzlib-devel >= 1.2, openssl-devel, curl-devel, expat-devel %{!?_without_docs:, xmlto, asciidoc > 6.0.3}\n BuildRoot:\t%{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)\n-Requires:\tgit-core, git-svn, git-cvs, git-arch, git-email, gitk, git-gui, perl-Git\n+\n+Requires:\tgit-core = %{version}-%{release}\n+Requires:\tgit-svn = %{version}-%{release}\n+Requires:\tgit-cvs = %{version}-%{release}\n+Requires:\tgit-arch = %{version}-%{release}\n+Requires:\tgit-email = %{version}-%{release}\n+Requires:\tgitk = %{version}-%{release}\n+Requires:\tgit-gui = %{version}-%{release}\n+Requires:\tperl-Git = %{version}-%{release}\n \n %description\n Git is a fast, scalable, distributed revision control system with an\n@@ -172,6 +180,9 @@ rm -rf $RPM_BUILD_ROOT\n %{!?_without_docs: %doc Documentation/technical}\n \n %changelog\n+* Sun Jan 06 2008 James Bowes \n+- Make the metapackage require the same version of the subpackages.\n+\n * Wed Dec 12 2007 Junio C Hamano \n - Adjust htmldir to point at /usr/share/doc/git-core-$version/\n \n-- \n1.5.4.rc2.1141.g437b09\n"},{"id":"64603","messageId":"7vprwe4s8e.fsf@gitster.siamese.dyndns.org","threadId":"11499","inReplyTo":"20080106173501.GB9349@spitfire","subject":"Re: [PATCH] Make the git metapackage require the same version of the subpackages.","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2008-01-06T20:24:49Z","receivedAt":"2008-01-06T20:24:49Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"James Bowes writes:\n\n> Without explicit version deps in the rpm spec file, 'yum update git'\n> effectively does nothing. Require explicit versions of the subpackages, so that\n> they get pulled in on an update.\n>\n> Signed-off-by: James Bowes \n\nI am asking as an RPM illiterate, not questioning the validity\nof what your patch does.\n\nThe approach your patch takes feels like the right way we should\nhave taken from the beginning. Does this supersede the \"fix\" in\n5587cac28be66acf5edc2a4b83b67c8cfffbc5e9 (GIT 1.5.3.1: obsolete\ngit-p4 in RPM spec file)? IOW, if we had Requires for the same\nversion from the beginning, we wouldn't have had the problem\nwhen we dropped git-p4 package?\n"},{"id":"64605","messageId":"3f80363f0801061313o514fa01bje354503483db47ab@mail.gmail.com","threadId":"11499","inReplyTo":"7vprwe4s8e.fsf@gitster.siamese.dyndns.org","subject":"Re: [PATCH] Make the git metapackage require the same version of the subpackages.","fromName":"James Bowes","fromEmail":"jbowes@dangerouslyinc.com","sentAt":"2008-01-06T21:13:26Z","receivedAt":"2008-01-06T21:13:26Z","isPatch":true,"sender":{"key":"jbowes@dangerouslyinc.com","avatar":"https://gravatar.com/avatar/a2fe98c66b2b47a9fa9d2ba92ff949d54c3208b1f8acc2e745b4b84ae3c4483a?d=mp&s=160"},"body":"On Jan 6, 2008 3:24 PM, Junio C Hamano wrote:\n> James Bowes writes:\n>\n> > Without explicit version deps in the rpm spec file, 'yum update git'\n> > effectively does nothing. Require explicit versions of the subpackages, so that\n> > they get pulled in on an update.\n> >\n> > Signed-off-by: James Bowes \n>\n> I am asking as an RPM illiterate, not questioning the validity\n> of what your patch does.\n>\n> The approach your patch takes feels like the right way we should\n> have taken from the beginning. Does this supersede the \"fix\" in\n> 5587cac28be66acf5edc2a4b83b67c8cfffbc5e9 (GIT 1.5.3.1: obsolete\n> git-p4 in RPM spec file)? IOW, if we had Requires for the same\n> version from the beginning, we wouldn't have had the problem\n> when we dropped git-p4 package?\n\nI believe the obsolete is still needed, as you'd need a way to tell\nrpm to just get rid of git-p4 entirely.\n\n-James\n"},{"id":"64607","messageId":"7vhchq4pgx.fsf@gitster.siamese.dyndns.org","threadId":"11499","inReplyTo":"3f80363f0801061313o514fa01bje354503483db47ab@mail.gmail.com","subject":"Re: [PATCH] Make the git metapackage require the same version of the subpackages.","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2008-01-06T21:24:30Z","receivedAt":"2008-01-06T21:24:30Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"\"James Bowes\" writes:\n\n> I believe the obsolete is still needed, as you'd need a way to tell\n> rpm to just get rid of git-p4 entirely.\n\nThanks.\n\nI am also wondering what should happen to spec file if we were\nto later re-introduce git-p4, but that is not an immediate\nconcern.\n"}]}