public inbox for cygwin-apps@cygwin.com
 help / color / mirror / Atom feed
From: Ken Brown <kbrown@cornell.edu>
To: cygwin-apps@cygwin.com
Subject: [PATCH cygport 0/4] Support dependencies with version relations
Date: Mon, 25 Jun 2018 15:10:00 -0000	[thread overview]
Message-ID: <20180625151031.13164-1-kbrown@cornell.edu> (raw)

This patch series presupposes the name change of DEPEND to
BUILD_DEPENDS proposed by Jon in
https://sourceware.org/ml/cygwin-apps/2017-10/msg00018.html.

Ever since setup began using libsolv, setup (and calm) are able to
support dependencies with version relations.  These can be specified
in the 'depends:' field of a .hint file like this:

depends: cygwin (>=2.10), bash (=4.4.12-3)

This patch series adds cygport support for such dependencies.  The
user can specify versioned dependencies via a new [PKG_]DEPENDS
variable, and cygport generates versioned dependencies in its
auto-generated .hint files.  The latter is intended to avoid problems
like the one reported here:

  https://cygwin.com/ml/cygwin/2018-03/msg00365.html

An obvious potential problem is the confusion between the new DEPENDS
and the old DEPEND (which is an obsolete alias for BUILD_DEPENDS after
Jon's renaming).  I've chosen to ignore that problem in these patches,
except for a warning in the documentation.  At the very least, a
HEADSUP email to maintainers will be required.  Alternatively, we
could drop DEPENDS and insist on PKG_DEPENDS even if there's only one
package.


Ken Brown (4):
  pkg_dist: use 'depends:' instead of 'requires:' in generated .hint
    files
  pkg_dist: add support for dependencies with version relations
  pkg_dist: record package versions in "depends:"
  Update sample.cygport

 data/sample.cygport | 10 ++++-
 lib/pkg_pkg.cygpart | 91 +++++++++++++++++++++++++++++++++++++++------
 2 files changed, 88 insertions(+), 13 deletions(-)

-- 
2.17.0

             reply	other threads:[~2018-06-25 15:10 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-06-25 15:10 Ken Brown [this message]
2018-06-25 15:10 ` [PATCH cygport 2/4] pkg_dist: add support for " Ken Brown
2018-06-25 15:10 ` [PATCH cygport 3/4] pkg_dist: record package versions in "depends:" Ken Brown
2018-06-25 15:10 ` [PATCH cygport 4/4] Update sample.cygport Ken Brown
2018-06-25 15:10 ` [PATCH cygport 1/4] pkg_dist: use 'depends:' instead of 'requires:' in generated .hint files Ken Brown

Reply instructions:

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

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

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20180625151031.13164-1-kbrown@cornell.edu \
    --to=kbrown@cornell.edu \
    --cc=cygwin-apps@cygwin.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).