public inbox for [email protected]  

help / color / mirror / Atom feed

From: Bradford Boyle <[email protected]>
To: [email protected]
Subject: Updated packaging for MobilityDB 1.1.0~beta1
Date: Sat, 4 Nov 2023 16:19:05 -0700
Message-ID: <CAOMoQbTOZ0ZLUnsu2Z8NJkS3JrjKDpbYgfWWn0KD_jGBJHnj6g@mail.gmail.com> (raw)

Hello,

I have just pushed an update to the Debian packaging of MobilityDB to
salsa.debian.org [1] that addresses the feedback [2] from the first
attempt at packaging this extension. The build passes the GitLab CI
pipeline and I have verified that the package builds and autopkgtests
pass on arm64 (via sbuild). Below is a summary of how I have addressed
the comments/questions raised in the previous review. As time permits, I
would appreciate another review.

* I have updated debian/copyright to more accurately reflect the
  copyrights found in the source. I bootstrapped this file using cme[3]
  and then manually addressed all of the TODOs. I've also ensured that
  lintian doesn't raise any errors or warnings related to d/copyright.
* I've updated d/tests/installcheck to avoid the use of
  shared_preload_libraries when creating the test cluster
    * Tests instead set session_preload_libraries via PGOPTIONS to load
      the postgis extension. I don't think it is possible to link
      against postgis since it is not a shared library (no SONAME).
* I've updated d/control.in so that PGVERSION is no longer used --
  declaring a dependency on postgresql-postgis is sufficient

Thanks,

-- Bradford


[1]: https://salsa.debian.org/postgresql/mobilitydb/-/commit/9dfff5eee4eab17b8d20ecd408ed10b830750244
[2]: https://www.postgresql.org/message-id/ZMfMWcLtZIVxbDYj%40msg.df7cb.de
[3]: https://wiki.debian.org/CopyrightReviewTools#cme

view thread (18+ messages)  latest in thread

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: [email protected]
  Cc: [email protected], [email protected]
  Subject: Re: Updated packaging for MobilityDB 1.1.0~beta1
  In-Reply-To: <CAOMoQbTOZ0ZLUnsu2Z8NJkS3JrjKDpbYgfWWn0KD_jGBJHnj6g@mail.gmail.com>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox