On 05/29/12 17:42, Brock Pytlik wrote:
On 05/29/12 15:30, Shawn Walker wrote:
On 05/09/12 19:10, Brock Pytlik wrote:
Webrev:
https://cr.opensolaris.org/action/browse/pkg/bpytlik/7140060-v2
Bug:
7140060 install and update should allow multiple pattern matches when
they intersect
src/modules/client/api_errors.py:
lines 577-578: I think many users may stumble over the phrase
"disjoint set". Perhaps something simpler like this:
The following patterns specify different versions of the same package(s):
That works. Thanks for the wording.
src/modules/client/imageplan.py:
line 3976: please add a brief comment noting that fmri publisher
filtering is handled later
Sure
line 4094: s/specificed/specified/
Thanks.
lines 4094-4117: It looks like if the user specifies a version of an
installed package that is not available from the sticky publisher,
that this may cause a multispec error to be raised, which doesn't seem
quite right. Is that right?
Is this the situation you're talking about?
Image has two publishers configured, test1 and test2, test1 is higher
ranked. Test2 is sticky and package foo@1 is installed from test2. foo@2
is available from test1 only. The user types 'pkg update foo@2' or 'pkg
install foo@2'?
If that's the case, here's the error that's produced:
| $ /tmp/ips.test.18885/fakeroot/pkg -R /tmp/ips.test.18885/0/image0
--debug smf_cmds_dir=/tmp/ips.test.18885/0/smf_cmds update a@2
|
| pkg update: The following patterns only matched packages that are from
publishers other than that which supplied the already installed version
of this package
| a@2: test
Which is the wrong_publisher error, which seems like the right error to
me. (The same error appears if install is used instead of update.)
Notice that in the code you mentioned, stripped_by_publisher is set to
true on line 4110. That means the code on lines 4116 and 4117 is used
(assuming no packages match the pattern), so we get the wrong_publisher
error.
If I haven't covered the situation you were concerned about, please let
me know and I'll try again.
Ah, ok, I didn't correctly understand what would happen. That is the
result I would expect.
Do these changes need to be made to any of our other pattern matching
code? (Sadly...) Those can be separate RFEs of course.
Ship it,
-Shawn
_______________________________________________
pkg-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/pkg-discuss
