[Bug 227646] Review Request: grass - GRASS (Geographic Resources Analysis Support System)

[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: grass -  GRASS (Geographic Resources Analysis Support System)


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





------- Additional Comments From cbalint at redhat.com  2007-03-15 10:03 EST -------
updated.
Spec URL: http://openrisc.rdsor.ro/grass.spec
SRPM URL: http://openrisc.rdsor.ro/grass-6.2.1-11.src.rpm

(In reply to comment #32)
> Created an attachment (id=150081)
 --> (https://bugzilla.redhat.com/bugzilla/attachment.cgi?id=150081&action=view) [edit]
> rpmlint log for grass-6.2.1-10
> 
> Well, for 6.2.1-10:
> 
> * Requies:
>   A. for -devel:
>      * lesstiff-devel - perhaps typo..
lesstif-devel is the correct. (according to one of include file) 

> * rpmlint (attached)
>   - Still complains about "undefined-non-weak-symbol", perhaps
exported LDFLAGS='-lm' for now globaly.
I would like to investigate it more deep and fix it upstream.

>       However, the better idea is to add gdal.pc in gdal-devel,
>       which has:
> ---------------------------------------------------------
> prefix=/usr
> exec_prefix=/usr
> libdir=/usr/lib
> includedir=/usr/include
> 
> Name: GDAL
> Description: GIS file format library
> Version: 1.4.0
> Libs: -L${libdir}
> Cflags: -I%{includedir}/gdal
> ---------------------------------------------------------
>     , which is installed under %{_libdir}/pkgconfig, and fix
>       grass.pc so that
> ---------------------------------------------------------
> # Package Information for pkg-config
> prefix=/usr
> exec_prefix=/usr
> libdir=/usr/lib
> includedir=/usr/include
> 
> Name: GRASS
> Description: GRASS GIS
> Version: 6.2.1
> Libs: -L%{libdir} -lgrass_I -lgrass_vask -lgrass_gmath -lgrass_gis
> -lgrass_datetime -lgrass_gproj -lgrass_vect -lgrass_dbmibase -lgrass
> _dbmiclient
> Cflags: -I${includedir}/grass
> Requires: xt x11 xproto glproto gdal
> ---------------------------------------------------------
>     (on i386, ppc!! on x86_64, $libdir differ)

   I followed second idea, added olso to gdal .pc file. I would like post olso 
upstream. (Still work to update gdal, so it may take a while to propagate 
into -extras)

  Thank you for the fix !
 
> * For keeping timestamps:
>   - For this the following seems to work (check how the timestamp
>     change by adding following).
 
> sed -i 's|^cp |cp -p |' tools/build_html_index.sh <- ADD
> sed -i 's|-cp |-cp -p |' Makefile <- ADD
> %build
> export INSTALL="%{__install} -c -p" <- ADD

Added.
  Thanks again for the hint !
 
> ---------------------------------------------------------
> 
> * For license:
>   - Well, I just asked you and from your comment it seems no problem

Well, its fine.


-- 
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.