1 |
On 09/18/2015 01:14 PM, Alexis Ballier wrote: |
2 |
> On Fri, 18 Sep 2015 13:04:45 +0200 |
3 |
> hasufell <hasufell@g.o> wrote: |
4 |
> |
5 |
>> On 09/18/2015 12:56 PM, Alexis Ballier wrote: |
6 |
>>> On Fri, 18 Sep 2015 11:58:09 +0200 |
7 |
>>> hasufell <hasufell@g.o> wrote: |
8 |
>>> |
9 |
>>>> On 09/18/2015 11:55 AM, Duncan wrote: |
10 |
>>>>> Alexis Ballier posted on Fri, 18 Sep 2015 11:04:19 +0200 as |
11 |
>>>>> excerpted: |
12 |
>>>>> |
13 |
>>>>>>> Keep in mind what this implies when you change these |
14 |
>>>>>>> dependencies without bumping the ebuilds that use them. |
15 |
>>>>>> |
16 |
>>>>>> only way i see these changing is with a new ros_messages_*** |
17 |
>>>>>> useflag, which will cause a rebuild anyway |
18 |
>>>>> |
19 |
>>>>> ?? Only with --newuse or similar, tho? Otherwise USE (or |
20 |
>>>>> USE_EXPAND here) changes don't trigger rebuilds, do they? |
21 |
>>>>> |
22 |
>>>> |
23 |
>>>> Correct. It's not good to rely on this and expect users to have a |
24 |
>>>> certain update pattern or even use a particular PM. |
25 |
>>>> |
26 |
>>> |
27 |
>>> then they wont have the messages if they don't rebuild, and |
28 |
>>> cat/pkg[ros_messages_newthing] deps wont be satisfied and the pm |
29 |
>>> will do the right thing by rebuilding. |
30 |
>>> nothing to worry about, really. |
31 |
>>> |
32 |
>> |
33 |
>> cat/pkg[ros_messages_newthing] will not be in users VDB until you |
34 |
>> revbump cat/pkg, so I don't think it's that easy. |
35 |
>> |
36 |
> |
37 |
> and i think you're confused about what dynamic deps is and is not: |
38 |
> cat/pkg[ros_messages_newthing] is not in vdb, so is not satisfied, so |
39 |
> pm tries to find something that satisfies it from tree or dies. |
40 |
> |
41 |
|
42 |
I'm not confused about dynamic deps, but about your example, because you |
43 |
didn't say anything about revbumps. |
44 |
|
45 |
If cat/pkg is not (rev)bumped, then there will be no check about whether |
46 |
cat/pkg[ros_messages_newthing] is satisfied in the first place. |
47 |
|
48 |
That was my point. |