1 |
On Wed, Aug 27, 2008 at 10:38 AM, Fabian Groffen <grobian@g.o> wrote: |
2 |
> On 27-08-2008 10:28:57 -0700, Robin H. Johnson wrote: |
3 |
>> On Wed, Aug 27, 2008 at 06:35:57PM +0200, Fabian Groffen wrote: |
4 |
>> > For that reason I'd pretty much prefer to keep the CVS Header in place, |
5 |
>> > unless there is a very good reason to remove it. |
6 |
>> As I wrote in the other thread, my reason for asking is that it's one of |
7 |
>> the things that doesn't have clear mapping in the Git world. As a side |
8 |
>> benefit, getting rid of it also makes the double-commit mess go away. |
9 |
> |
10 |
> For who is it a mess? Not for repoman users, I suppose, and everyone |
11 |
> should be using it, right? As the one who personally played with the |
12 |
> code in repoman that determines whether or not the "double commit" is |
13 |
> necessary, I think it's mostly a repoman internal problem. The commit |
14 |
> script problems put aside. |
15 |
|
16 |
So you are saying we should do what? |
17 |
|
18 |
precompute the CVS header and inject it into $header$ ourselves |
19 |
take the checksums |
20 |
generate the manifest |
21 |
revert the $header$ change |
22 |
then commit the ebuild and manifest at once |
23 |
|
24 |
? |
25 |
|
26 |
The only reason we have double commits right now is that the $header$ |
27 |
replacement is done by cvs at commit time so if we don't do two |
28 |
commits the checksums all break due to the substitution..how is that |
29 |
repoman's fault? |
30 |
|
31 |
> |
32 |
>> For your use case, it should be possible to just ask Git for updates to |
33 |
>> the given directory, and apply those to your own tree. |
34 |
> |
35 |
> Another VCS is another story. If we're switching, it would be nice if |
36 |
> the notion of overlays shadowing the main tree would be taken into |
37 |
> account. Especially since I don't think Prefix will "merge" any time |
38 |
> soon, but we are plagued by the thing called "growth". |
39 |
> |
40 |
> |
41 |
> -- |
42 |
> Fabian Groffen |
43 |
> Gentoo on a different level |
44 |
> |
45 |
> |