[Bug 232792] Review Request: mapserver - Environment for building spatially-enabled internet applications

[bugzilla at redhat.com]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: mapserver - Environment for building spatially-enabled internet applications


https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=232792





------- Additional Comments From cbalint at redhat.com  2007-04-24 13:05 EST -------

>     * In your opinion it is the bug of php side that %{_libdir}/php4
>       is not owned by any package?

It _should be owned by:
[cbalint at cbalint ~]$ rpm -qf /usr/lib64/php
php-common-5.2.1-5

So i should _not_ own it !

>     * And what does php"4" means? This "4" is of no relation with
SHIIT.... 
Sorry, ok i should remove 4, and put all in 
/usr/lib64/php/modules/
Seems thigs changed since a while ...

>       php version (currently 5.2.1)?
Ya right, sorry for confusion.

>     * And as far as I saw some php modules rpms, php modules (which
>       I think so) are installed under %{_libdir}/php/modules/, and this
>       directory (%{_libdir}/php/modules) is owned by php-common.
correct !
 
> * perl .so module permission
>   - Well, actually you fixed the permission by:
> ----------------------------------------------------------
> %attr(0755,root,root) %{perl_vendorarch}/auto/mapscript/*
> ----------------------------------------------------------
>     However, this method leaves the following message which
>     I don't know I can ignore:
> ----------------------------------------------------------
> + /usr/lib/rpm/check-buildroot
> + /usr/lib/rpm/redhat/brp-compress
> + /usr/lib/rpm/redhat/brp-strip-static-archive /usr/bin/strip
> + /usr/lib/rpm/redhat/brp-strip-comment-note /usr/bin/strip /usr/bin/objdump
> /usr/bin/strip: unable to copy file
> '/var/tmp/mapserver-4.10.1-3.fc7-root-mockbuild/usr/lib/perl5/vendor_perl/5.8.8/i386-linux-thread-multi/auto/mapscript/mapscript.so'
> reason: Permission denied
> + /usr/lib/rpm/brp-python-bytecompile
> + /usr/lib/rpm/redhat/brp-java-repack-jars
> ----------------------------------------------------------
>    To avoid this, it seems that the permission of mapscript.so must
>    be changed to 0755 at the install stage, not by setting attr.
> 
> * documentation
> ----------------------------------------------------------
>  %files
>  %defattr(-,root,root)
> -%doc COMMITERS GD-COPYING HISTORY.TXT INSTALL
> -%doc MIGRATION_GUIDE.TXT README README.CONFIGURE
> +%doc COMMITERS GD-COPYING HISTORY.TXT 
> +%doc INSTALL MIGRATION_GUIDE.TXT
> ----------------------------------------------------------
>   - While ReADME.CONFIGURE is not needed, IMO README should be
>     left as documentation.

Yay, i did stupid F6 instead of F5 at a point.

Ok i fix all issues and upload things for tomorrow.

(head out for now in a rush)

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.