[HEADS UP] rpm-4.12.90 in rawhide

[Florian Festi]

On 07/27/2015 04:55 PM, Lubos Kardos wrote:
> The current behavior is right. Your install script insert files into buildroot
> and these files are not listed in %files section.
> 
> Correctly you should replace this:
> 
> %files
> %doc AUTHORS COPYING NEWS README
> ...
> 
> with this:
> 
> %files
> %{_pkgdocdir}/AUTHORS
> %{_pkgdocdir}/COPYNIG
> %{_pkgdocdir}/NEWS
> %{_pkgdocdir}/README
> %{_pkgdocdir}/ChangeLog
> %{_pkgdocdir}/TODO
> ...
> 
> Tag %doc followed by relative path of a file is used when you want rpm to copy
> files from BUILD to BUILDROOT document directory but in your case this is done
> by your install script.
> 
> Lubos


>> there is another change involving %doc. A piece of libdvdread.spec: "
>> %files
>> %defattr(-,root,root,-)
>> %doc AUTHORS COPYING NEWS README
>> %{_libdir}/libdvdread.so.*
>>
>> %files devel
>> %defattr(-,root,root,-)
>> %doc ChangeLog TODO
>> ...

RPM actually copies those to
/builddir/build/BUILDROOT/php-pecl-imagick-3.1.2-3.fc24.i386/usr/share/doc/php-pecl-imagick-devel/
and includes them into the devel package.

It now refuses to use the copies in
/builddir/build/BUILDROOT/php-pecl-imagick-3.1.2-3.fc24.i386/usr/share/doc/php-pecl-imagick
which have previously been added to the main package erroneously.

Florian

-- 

Red Hat GmbH, http://www.de.redhat.com/ Registered seat: Grasbrunn,
Commercial register: Amtsgericht Muenchen, HRB 153243,
Managing Directors: Charles Cachera, Michael Cunningham, Michael
O'Neill, Charles Peters