*** Bug 178162 has been marked as a duplicate of this bug. ***

(In reply to comment #0)
> Spec URL: http://openrisc.rdsor.ro/libgeotiff.spec
> SRPM URL: http://openrisc.rdsor.ro/libgeotiff-1.2.4rc1.fc8.src.rpm

Both seems 404?

(A note:
 For pre-versions like rc, the EVR of rpm must be like:
 1.2.4-0.X.rcY to avoid broken EVR path.)

I am sorry.

re-uploaded.

Spec URL: http://openrisc.rdsor.ro/libgeotiff.spec
SRPM URL: http://openrisc.rdsor.ro/libgeotiff-1.2.4-0.1.rc1.fc8.src.rpm

EPSG dataset license is OK now.

Confirmed by Thomas Callaway <tcallawa>, olso FSF.

   Minor thing is that EPSG dataset is splitted in csv files, see
ChangeLog but without alter the data essence itself, just 
reorganise original EPSG wich comes as SQl dumps or MS Access 
files.
   EPSG hates alter of dataset so I must be carefull looking at that.

At a point my plan is to split separate epsg-data.noarch one and re-base
gdal and geotiff over it. Thats should be clearest way include EPSG 
dataset, and trace down how csv files are generated.

~cristian

Created attachment 159804 [details]
mock build log of libgeotiff 1.2.4-0.1.rc1 on rawhide i386

Very rapid comment because I want to go to bed..

* debuginfo rpm is of no use. It seems that -g option is
  removed.
* I suggest to move the header files to the subdirectory of /usr/include
  and hide them from /usr/include (as you wrote on .pc file)
* Specify the full URL of the source.
* I may be wrong, however (for example)
  /usr/include/geonames.h contains
--------------------------------------------
    47	static KeyInfo _keyInfo[] =  {
    48	#   include "geokeys.inc"   /* geokey database */
    49	    END_LIST
--------------------------------------------
  but geokeys.inc is missing?
* /usr/include/xtiffio.h contains
--------------------------------------------
    10	#include "tiffio.h"
--------------------------------------------
  And it seems that tiffio.h is included in libtiff-devel.
  So libgeotiff-devel should require libtiff-devel.
* Check if
--------------------------------------------
make install DESTDIR=<foo> INSTALL="%{__install} -p"
--------------------------------------------
  works to keep timestamps on the header files installed.

Spec URL: http://openrisc.rdsor.ro/libgeotiff.spec
SRPM URL: http://openrisc.rdsor.ro/libgeotiff-1.2.4-0.2.rc1.fc8.src.rpm

* Mon Jul 23 2007 Balint Cristian <cbalint> 1.2.4-0.2.rc1
- fix debuginfo usability
- move header files to the subdirectory
- specify the full URL of the source
- leave *.inc headers included
- libgeotiff-devel should require libtiff-devel
- works to keep timestamps on the header files installed

- docs file proper triage

Now debuginfo rpm is correct.
For 1.2.4-0.2:

* License
  - Under which is the whole package of libgeotiff licensed ?
    Public domain or MIT? As far as I checked the source codes,
    many codes are under MIT (X).

* pkgconfig .pc file
  - Currently pkg-config returns:
--------------------------------------------------
$ pkg-config --cflags libgeotiff
-I/usr/include/libgeotiff/libgeotiff 
--------------------------------------------------

* Directory ownership issue
--------------------------------------------------
$ LANG=C rpm -qf /usr/share/epsg_csv/alias.csv 
libgeotiff-1.2.4-0.2.rc1.fc8
$ LANG=C rpm -qf /usr/share/epsg_csv/ 
file /usr/share/epsg_csv is not owned by any package
--------------------------------------------------

(In reply to comment #5)
> At a point my plan is to split separate epsg-data.noarch one and re-base
> gdal and geotiff over it. Thats should be clearest way include EPSG 
> dataset, and trace down how csv files are generated.
* A question:
  - Are there any other (splitted) source which contains only EPSG
    data or are you going to create epsg-data rpm as a subpackage of
    libgeotiff?
  - When epsg-data is splitted out, will epsg-data be needed only to rebuild
    libgeotiff or gdal, or will it be needed even for runtime?

Spec URL: http://openrisc.rdsor.ro/libgeotiff.spec
SRPM URL: http://openrisc.rdsor.ro/libgeotiff-1.2.4-0.3.rc1.fc8.src.rpm

* Tue Jul 24 2007 Balint Cristian <cbalint> 1.2.4-0.3.rc1
- codes are under MIT
- pkg-config cflags return fix
- epsg_csv ownership


RE for #5:

I would like to split out epsg data for 3 reasons:

- 1) would be nice to have noarch epsg dataset separatley so any time i can 
respin agains newer versions independently agains gdal/geotiff

- 2) this way i can include all _original_ dataset and people can see how it 
explode in .csv and .inc ones, EPSG loves to see we NOT alter their database 
just copy portions and re-arange tham, however their sql dumps arent usefull 
when you fopen() at all. SO license statement will be not brake at all.

- 3) gdal and geotiff with small patch can pick up tham on the fly from 
specific folder, and any respin off epsg would NOT imply rebuild of 
gdal/geotiff code, actualy they fopen and search through it.

   However i still discuss with Frank how splitting works i need to find a way 
for do it in %build section of files. It might be strange since now he use
postgresql query to dump in csv files, but in %build time i cannot do like 
this, so need to rewrite script wich simply parse sql file dumps the same 
way. Its hard but would be nice.
 

For epsg data:
- So, currently epsg data is included in libgeotiff tarball, however
  you mean that you (and Frank) want to create new tarball of epsg
  data and distribute epsg data tarball and libgeotiff tarball seperately,
  am I correct?

  In such case, when splitting is done, you have to create a new review
  request for epsg data (and for now I want to proceed the review of
  libgeotiff as it is).

1) It will take a while.
2) Yes I will subbmit new FE-REW one.
3) For now lets follow what we have at all, i agree.

Okay.

--------------------------------------------------------
   This package (libgeotiff) is APPROVED by me
--------------------------------------------------------

Please write the CVS Request according to 
http://fedoraproject.org/wiki/PackageMaintainers/CVSAdminProcedure

New Package CVS Request
=======================
Package Name: libgeotiff
Short Description: GeoTIFF format library
Owners: cbalint
Branches: FC-6 F-7 EL-4 EL-5
InitialCC:

libgeotiff-1.2.4-0.3.rc1.fc7 has been pushed to the Fedora 7 stable repository.  If problems still persist, please make note of it in this bug report.