public inbox for cygwin-apps@cygwin.com
 help / color / mirror / Atom feed
From: Ken Brown <kbrown@cornell.edu>
To: cygwin-apps@cygwin.com
Subject: Re: [setup topic/libsolv] Packages contained in multiple repositories
Date: Sat, 21 Oct 2017 20:21:00 -0000	[thread overview]
Message-ID: <02a981d0-1b57-0b49-2a72-cf6241658b78@cornell.edu> (raw)
In-Reply-To: <16655ef1-6672-b071-29d2-e195c8411ba7@dronecode.org.uk>

On 10/19/2017 9:36 AM, Jon Turney wrote:
> On 18/10/2017 17:41, Achim Gratz wrote:
>> Ken Brown writes:
>>> In retrospect, I'm not sure this patch is right, but I'm sending it
>>> anyway for the sake of discussion.  My hesitation comes from the fact
>>> that libsolv might have a good reason for preferring the one it chose,
>>> e.g., if we've assigned priorities to the repos.  On the other hand,
>>> if we've gone to the trouble of assigning priorities, shouldn't
>>> packagemeta reflect our choice?
>>
>> Extrapolating from my experience with zypper, libsolv should stick with
>> the repo the installed package comes from even if some other repo has a
>> newer version.
> 
> Unfortunately, we are limited by the fact that installed.db doesn't 
> record which repo an installed package came from.
> 
> We map repo setup.ini release: labels to package vendor names, so we 
> assume it's 'Cygwin' for an installed package (package_db.cc:115), and 
> run the solver with SOLVER_FLAG_ALLOW_VENDORCHANGE on (libsolv.cc:745) 
> to allow it to get upgraded by a package in the repo it actually came 
> from, but we've forgotten.
> 
> I'm not overly concerned about this: we don't define what happens in 
> this situation currently, and if the packages are identical, it's no 
> problem.

Here's a situation where I think it might be a problem: 
fixup_source_package_ids only considers packageversions that are 
registered in packagedb.  If libsolv later selects a packageversion from 
a different repo and the user has requested the source, it might not be 
found.

Ken

      parent reply	other threads:[~2017-10-21 20:21 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-10-18 14:15 Ken Brown
2017-10-18 16:41 ` Achim Gratz
2017-10-19 13:36   ` Jon Turney
2017-10-19 17:14     ` Achim Gratz
2017-10-21 20:21     ` Ken Brown [this message]

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=02a981d0-1b57-0b49-2a72-cf6241658b78@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).