Git development
 help / color / mirror / Atom feed
* rpmbuild fails
@ 2006-10-10 20:31 Anders Larsen
  2006-10-10 22:18 ` Josef Weidendorfer
  0 siblings, 1 reply; 5+ messages in thread
From: Anders Larsen @ 2006-10-10 20:31 UTC (permalink / raw)
  To: git

The current git fails when building rpms:

	RPM build errors:
	    Installed (but unpackaged) file(s) found:
	   /usr/lib/perl5/vendor_perl/5.8.6/Git.pm

Indeed git.spec.in does not specify any files for perl-Git, only
	%files -n perl-Git -f perl-files
	%defattr(-,root,root)
nothing more.

How is this supposed to be handled correctly?

Cheers
 Anders

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: rpmbuild fails
  2006-10-10 20:31 rpmbuild fails Anders Larsen
@ 2006-10-10 22:18 ` Josef Weidendorfer
  2006-10-11  0:38   ` [RFH] RPM gurus? (Re: rpmbuild fails) Junio C Hamano
  0 siblings, 1 reply; 5+ messages in thread
From: Josef Weidendorfer @ 2006-10-10 22:18 UTC (permalink / raw)
  To: Anders Larsen; +Cc: git, Junio Hamano

On Tuesday 10 October 2006 22:31, Anders Larsen wrote:
> The current git fails when building rpms:
> 
> 	RPM build errors:
> 	    Installed (but unpackaged) file(s) found:
> 	   /usr/lib/perl5/vendor_perl/5.8.6/Git.pm

I got the same.
I'm no rpm guru, but changing %{perl_vendorarch} into
%{perl_vendorlib} in git.spec.in helped for me.

Obviously, the generated perl/Makefile installs Git.pm
into the perl vendor *library* directory, and not the arch
one.

I do not provide a real patch, as I have no idea if
this is really the correct fix (we have no arch-dependent
perl files anymore, yes?).

However, something like the above should go into git-1.4.3.

> Indeed git.spec.in does not specify any files for perl-Git, only
> 	%files -n perl-Git -f perl-files
> 	%defattr(-,root,root)
> nothing more.

Hmm... Obviously the "... -f perl-files" includes all files
mentioned in file "perl-files" into perl-Git. The problem was
that without the above change this file was empty.

Josef

^ permalink raw reply	[flat|nested] 5+ messages in thread

* [RFH] RPM gurus? (Re: rpmbuild fails)
  2006-10-10 22:18 ` Josef Weidendorfer
@ 2006-10-11  0:38   ` Junio C Hamano
  2006-10-11  8:25     ` Junio C Hamano
  0 siblings, 1 reply; 5+ messages in thread
From: Junio C Hamano @ 2006-10-11  0:38 UTC (permalink / raw)
  To: Josef Weidendorfer; +Cc: git, Anders Larsen

Josef Weidendorfer <Josef.Weidendorfer@gmx.de> writes:

> On Tuesday 10 October 2006 22:31, Anders Larsen wrote:
>> The current git fails when building rpms:
>> 
>> 	RPM build errors:
>> 	    Installed (but unpackaged) file(s) found:
>> 	   /usr/lib/perl5/vendor_perl/5.8.6/Git.pm
>
> I got the same.
> I'm no rpm guru, but changing %{perl_vendorarch} into
> %{perl_vendorlib} in git.spec.in helped for me.
>
> Obviously, the generated perl/Makefile installs Git.pm
> into the perl vendor *library* directory, and not the arch
> one.

Thanks.

Also I noticed that we now build-require perl(Error); I have to
ask kernel.org people to install it.

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [RFH] RPM gurus? (Re: rpmbuild fails)
  2006-10-11  0:38   ` [RFH] RPM gurus? (Re: rpmbuild fails) Junio C Hamano
@ 2006-10-11  8:25     ` Junio C Hamano
  2006-10-11 12:20       ` Josef Weidendorfer
  0 siblings, 1 reply; 5+ messages in thread
From: Junio C Hamano @ 2006-10-11  8:25 UTC (permalink / raw)
  To: git

Junio C Hamano <junkio@cox.net> writes:

> Josef Weidendorfer <Josef.Weidendorfer@gmx.de> writes:
>
>> Obviously, the generated perl/Makefile installs Git.pm
>> into the perl vendor *library* directory, and not the arch
>> one.

Does this look reasonable?

-- >8 --
git.spec.in: perl subpackage is installed in perl_vendorlib not vendorarch

Signed-off-by: Junio C Hamano <junio@hera.kernel.org>
---
 git.spec.in |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/git.spec.in b/git.spec.in
index 6d90034..9b1217a 100644
--- a/git.spec.in
+++ b/git.spec.in
@@ -97,7 +97,7 @@ find $RPM_BUILD_ROOT -type f -name '*.bs
 find $RPM_BUILD_ROOT -type f -name perllocal.pod -exec rm -f {} ';'
 
 (find $RPM_BUILD_ROOT%{_bindir} -type f | grep -vE "arch|svn|cvs|email|gitk" | sed -e s@^$RPM_BUILD_ROOT@@)               > bin-man-doc-files
-(find $RPM_BUILD_ROOT%{perl_vendorarch} -type f | sed -e s@^$RPM_BUILD_ROOT@@) >> perl-files
+(find $RPM_BUILD_ROOT%{perl_vendorlib} -type f | sed -e s@^$RPM_BUILD_ROOT@@) >> perl-files
 %if %{!?_without_docs:1}0
 (find $RPM_BUILD_ROOT%{_mandir} $RPM_BUILD_ROOT/Documentation -type f | grep -vE "arch|svn|git-cvs|email|gitk" | sed -e s@^$RPM_BUILD_ROOT@@ -e 's/$/*/' ) >> bin-man-doc-files
 %else
-- 
1.4.3.rc2.gcc59

^ permalink raw reply related	[flat|nested] 5+ messages in thread

* Re: [RFH] RPM gurus? (Re: rpmbuild fails)
  2006-10-11  8:25     ` Junio C Hamano
@ 2006-10-11 12:20       ` Josef Weidendorfer
  0 siblings, 0 replies; 5+ messages in thread
From: Josef Weidendorfer @ 2006-10-11 12:20 UTC (permalink / raw)
  To: Junio C Hamano; +Cc: git

On Wednesday 11 October 2006 10:25, Junio C Hamano wrote:
> Does this look reasonable?
> 
> -(find $RPM_BUILD_ROOT%{perl_vendorarch} -type f | sed -e s@^$RPM_BUILD_ROOT@@) >> perl-files
> +(find $RPM_BUILD_ROOT%{perl_vendorlib} -type f | sed -e s@^$RPM_BUILD_ROOT@@) >> perl-files

That was exactly the modification I did, yes.
However, I can only say: It works for me (tm).
BTW, this is only Suse 10.1, x86.

Josef

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2006-10-11 12:20 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2006-10-10 20:31 rpmbuild fails Anders Larsen
2006-10-10 22:18 ` Josef Weidendorfer
2006-10-11  0:38   ` [RFH] RPM gurus? (Re: rpmbuild fails) Junio C Hamano
2006-10-11  8:25     ` Junio C Hamano
2006-10-11 12:20       ` Josef Weidendorfer

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox