| 1 |
On 11/29/2011 04:11 AM, "Paweł Hajdan, Jr." wrote: |
| 2 |
> On 11/26/11 11:29 AM, Sergei Trofimovich wrote: |
| 3 |
>> On Sat, 26 Nov 2011 10:15:53 +0100 |
| 4 |
>> ""Paweł Hajdan, Jr."" <phajdan.jr@g.o> wrote: |
| 5 |
>> |
| 6 |
>>> This shouldn't be allowed, should it? The package is keyworded only |
| 7 |
>>> ~x86, and I'm adding amd64 stable keyword, and repoman doesn't complain. |
| 8 |
>> |
| 9 |
>> It would need to fetch latest CVS file from server to check against previous state. |
| 10 |
>> I don't think repoman does any network IO safe updating metadata.dtd. |
| 11 |
> |
| 12 |
> Why does it warn about going straight to stable for _new_ ebuilds? Is it |
| 13 |
> because it knows locally that it's a new file, and because of that |
| 14 |
> doesn't need to connect to network? That'd make sense. |
| 15 |
|
| 16 |
Right, repoman parses the files in the CVS directories to figure that out. |
| 17 |
|
| 18 |
> Would it be possible to add repoman checks (possibly behind a special |
| 19 |
> flag) that could use "cvs diff" results? |
| 20 |
|
| 21 |
Sure, we can add an --allow-network option. That would also be useful |
| 22 |
for checking SRC_URI in this bug: |
| 23 |
|
| 24 |
https://bugs.gentoo.org/show_bug.cgi?id=315243 |
| 25 |
|
| 26 |
We could also add fine-grained options if you want to enable/disable |
| 27 |
specific checks that use the network and not others. |
| 28 |
-- |
| 29 |
Thanks, |
| 30 |
Zac |
Archives