threads / patch / 1820

patchUpdate cogito.spec.in

Subject: [PATCH] Update cogito.spec.in

## tl;dr

5 messages between Sep 16, 2005 and Sep 16, 2005. Diffs are folded; open one to read it.

replies: 4people: 3as markdown or json

Chris Wright· Sep 16, 2005, 06:47 UTC · lore

Latest updates to cogito.spec.in generated from review for Fedora Extras.

Signed-off-by: Chris Wright <chrisw@osdl.org>

---

 cogito.spec.in |   18 ++++++++++++------
 1 files changed, 12 insertions(+), 6 deletions(-)

e0ca49e6c375a68b3e4b3edfff752fef2cf585f6

Show changes to cogito.spec.in +12 −6

diff --git a/cogito.spec.in b/cogito.spec.in
--- a/cogito.spec.in
+++ b/cogito.spec.in
@@ -1,15 +1,14 @@
 Name: 		cogito
 Version: 	@@VERSION@@
 Release: 	1
-Vendor: 	Petr Baudis <pasky@suse.cz>
 Summary:  	The Cogito Version Control System
 License: 	GPL
 Group: 		Development/Tools
 URL: 		http://kernel.org/pub/software/scm/cogito/
 Source: 	http://kernel.org/pub/software/scm/cogito/%{name}-%{version}.tar.gz
-BuildRoot:	%{_tmppath}/%{name}-%{version}-root
-Prereq: 	sh-utils, diffutils, rsync, rcs, mktemp >= 1.5, git-core >= 0.99.3
-BuildArchitectures:	noarch
+BuildRoot:	%{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)
+Requires: 	git-core >= 0.99.3
+BuildArch:	noarch
 
 %description
 Cogito is a version control system layered on top of the git tree history
@@ -34,11 +33,18 @@ rm -rf $RPM_BUILD_ROOT
 %files
 %defattr(-,root,root)
 %{_bindir}/*
-%dir /usr/lib/cogito
-/usr/lib/cogito/*
+%dir %{_libdir}/cogito
+%{_libdir}/cogito/*
 %doc README COPYING Documentation/*
 
 %changelog
+* Thu Sep 15 2005 Chris Wright <chrisw@osdl.org> 0.14.1-1
+- Update to 0.14.1
+
+* Mon Aug 15 2005 Chris Wright <chrisw@osdl.org> 0.13-3
+- Update Buildroot, Requires and drop Vendor
+- use %{_libdir}
+
 * Wed Aug 10 2005 Pavel Roskin <proski@gnu.org> 0.13-1
 - Update summary and description
 - Make architecture-independent

Petr Baudis· Sep 16, 2005, 11:06 UTC · re: Chris Wright · lore

Re: [PATCH] Update cogito.spec.in

Dear diary, on Fri, Sep 16, 2005 at 08:47:24AM CEST, I got a letter where Chris Wright <chrisw@osdl.org> told me that...

Show 20 quoted lines

> e0ca49e6c375a68b3e4b3edfff752fef2cf585f6
> diff --git a/cogito.spec.in b/cogito.spec.in
> --- a/cogito.spec.in
> +++ b/cogito.spec.in
> @@ -1,15 +1,14 @@
>  Name: 		cogito
>  Version: 	@@VERSION@@
>  Release: 	1
> -Vendor: 	Petr Baudis <pasky@suse.cz>
>  Summary:  	The Cogito Version Control System
>  License: 	GPL
>  Group: 		Development/Tools
>  URL: 		http://kernel.org/pub/software/scm/cogito/
>  Source: 	http://kernel.org/pub/software/scm/cogito/%{name}-%{version}.tar.gz
> -BuildRoot:	%{_tmppath}/%{name}-%{version}-root
> -Prereq: 	sh-utils, diffutils, rsync, rcs, mktemp >= 1.5, git-core >= 0.99.3
> -BuildArchitectures:	noarch
> +BuildRoot:	%{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)
> +Requires: 	git-core >= 0.99.3
> +BuildArch:	noarch

Why did you remove all the stuff from Requires? They actually are ending up adding even trivial stuff like less to it in GIT.

-- 
				Petr "Pasky" Baudis
Stuff: http://pasky.or.cz/
If you want the holes in your knowledge showing up try teaching
someone.  -- Alan Cox

Chris Wright· Sep 16, 2005, 15:46 UTC · re: Petr Baudis · lore

Re: [PATCH] Update cogito.spec.in

* Petr Baudis (pasky@suse.cz) wrote:

Show 25 quoted lines

> Dear diary, on Fri, Sep 16, 2005 at 08:47:24AM CEST, I got a letter
> where Chris Wright <chrisw@osdl.org> told me that...
> > e0ca49e6c375a68b3e4b3edfff752fef2cf585f6
> > diff --git a/cogito.spec.in b/cogito.spec.in
> > --- a/cogito.spec.in
> > +++ b/cogito.spec.in
> > @@ -1,15 +1,14 @@
> >  Name: 		cogito
> >  Version: 	@@VERSION@@
> >  Release: 	1
> > -Vendor: 	Petr Baudis <pasky@suse.cz>
> >  Summary:  	The Cogito Version Control System
> >  License: 	GPL
> >  Group: 		Development/Tools
> >  URL: 		http://kernel.org/pub/software/scm/cogito/
> >  Source: 	http://kernel.org/pub/software/scm/cogito/%{name}-%{version}.tar.gz
> > -BuildRoot:	%{_tmppath}/%{name}-%{version}-root
> > -Prereq: 	sh-utils, diffutils, rsync, rcs, mktemp >= 1.5, git-core >= 0.99.3
> > -BuildArchitectures:	noarch
> > +BuildRoot:	%{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)
> > +Requires: 	git-core >= 0.99.3
> > +BuildArch:	noarch
> 
> Why did you remove all the stuff from Requires? They actually are ending
> up adding even trivial stuff like less to it in GIT.

Primary reason is it now requires git, which has those prereqs.

thanks, -chris

Junio C Hamano· Sep 16, 2005, 18:04 UTC · re: Chris Wright · lore

Re: [PATCH] Update cogito.spec.in

Chris Wright <chrisw@osdl.org> writes:

> Primary reason is it now requires git, which has those prereqs.

Which I fully agree with.

Could you help me to modernize the git-core.spec.in? I cannot even tell if it needs any fix (I am not an RPM guy), but I suspect dropping Vendor: may apply to git-core as well.

Chris Wright· Sep 16, 2005, 18:09 UTC · re: Junio C Hamano · lore

Re: [PATCH] Update cogito.spec.in

* Junio C Hamano (junkio@cox.net) wrote:

Show 9 quoted lines

> Chris Wright <chrisw@osdl.org> writes:
> 
> > Primary reason is it now requires git, which has those prereqs.
> 
> Which I fully agree with.
> 
> Could you help me to modernize the git-core.spec.in?  I cannot
> even tell if it needs any fix (I am not an RPM guy), but I
> suspect dropping Vendor: may apply to git-core as well.

Yes, I'm actually working on that ATM.  Update will follow shortly.

thanks, -chris