[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Re: Modern packaging for postgis



Hi Markus,

Thanks for the feedback.

On 10/10/2016 09:36 AM, Markus Wanner wrote:
> On 10/09/2016 12:02 AM, Sebastiaan Couwenberg wrote:
>> I've prepared a branch with changes to modernize the postgis packaging:
>>
>>  https://anonscm.debian.org/cgit/pkg-grass/postgis.git/log/?h=modern-packaging
> 
> Thanks for bringing this up again. Looks good from a first glance.
> However, I just moved to a new home and am unlikely to look at it in
> more detail within the next few days.
> 
> I gave this branch a spin on the pgapt Jenkins build farm.

Thanks, I'll have a look at the logs for postgis-source.

>> The goal of these changes is to bring the postgis package in line with
>> the other packages maintained within the Debian GIS team.
> 
> (As a side note, I'm still wondering if Debian GIS is the right team for
> a PostgreSQL extension. But modernized packaging is a good thing in any
> case.)

In general, no. The PostgreSQL team is more appropriate for PostgreSQL
extensions in general.

In the case of PostGIS, pgRouting and pg_comparator, these are
appropriate in Debian GIS, especially the former two which are OSGeo
projects. PostGIS is an essential project in the GIS ecosystem, and all
its GIS related dependencies (gdal, geos, proj & sfcgal) are all
maintained by the the Debian GIS team too. This was my motivation to
move postgis to the Debian GIS team three years ago. The GIS team is the
appropriate place to integrate GIS packages in Debian.

>> The control files are restructured with cme while preserving the changes
>> for PGDG. The separate lines for dependencies greatly increase
>> readability and gives cleaner diffs as a bonus.
> 
> Good. Agreed.
> 
>> The rules file is switched to use the dh sequencer to not need explicit
>> calls to the various dh_* utilities. This added dh_perl amongst others
>> for which the substitution variable was added to the -scripts package.
> 
> I'm worried about backwards-compatibility, here.
> 
> At least the current build already failed building the source package
> for wheezy, trusty and precise:
> 
> dpkg-source: error: syntax error in source/debian/control at line 27:
> continued value line not in field
> dpkg-buildpackage: error: dpkg-source -i -I --before-build source gave
> error exit status 25

This is indeed something that needs to be fixed, as postgis-binaries
built successfully on these distributions before.

>> I've verified that both binary-arch & binary-indep builds still work.
>> And also verified the build in a cowbuilder chroot with the PGDG
>> repository enabled, but I suspect that the Jenkins build environment may
>> differ somewhat.
> 
> It's hardly the build environment, but rather the fact that the package
> should build from the same sources on older Debian and Ubuntu releases.
> 
>> Is there some way to do experimental builds for PGDG which won't
>> automatically be published?
> 
> Yes, please ask myon@debian.org for access to that Jenkins builder.

Since Jenkins now uses SSO I can at least access the web interface which
was not possible before. I don't really need to upload to PGDG.

>> I'd like to merge these changes as soon as possible to get them into
>> stretch, but I'd like you to confirm that they cause any adverse effects
>> for PGDG as they did the first time I implemented modernized packaging
>> some time ago.
> 
> Thanks.

As said, I'll have a look at the failures on the older distributions,
those should be fixed.

Kind Regards,

Bas

-- 
 GPG Key ID: 4096R/6750F10AE88D4AF1
Fingerprint: 8182 DE41 7056 408D 6146  50D1 6750 F10A E88D 4AF1


Reply to: