| 1 |
-- Spider <spider@g.o> spake thusly: |
| 2 |
|
| 3 |
> begin quote |
| 4 |
> On Fri, 14 Mar 2003 06:19:17 -0700 |
| 5 |
> Alain Penders <alain@g.o> wrote: |
| 6 |
> |
| 7 |
>> > I'm forced to proofread the new builds completely as to avoid |
| 8 |
>> > getting messed over. |
| 9 |
> |
| 10 |
>> Maybe because they trust that all our developers know how to diff a |
| 11 |
>> submitted ebuild against the last approved one? |
| 12 |
> |
| 13 |
> What do you think I mean with proofread, really? |
| 14 |
> (vimdiff and gtkdiff are both quite handy on larger stuff , gdm comes |
| 15 |
> to mind as more or less a nightmareish example) |
| 16 |
> |
| 17 |
>> Even with a changelog entry, I would never add a user-submitted |
| 18 |
>> ebuild without diffing it and making sure I know what changed and |
| 19 |
>> why. |
| 20 |
> |
| 21 |
> of course not, I'm not inclined to have my or others systems |
| 22 |
> compromised or messed over, But my point is: adding a ChangeLog or |
| 23 |
> stating what is done difference does make a change when submitting a |
| 24 |
> build for something thats already in the tree. |
| 25 |
|
| 26 |
While it's an excellent point that users should submit Changelogs, it |
| 27 |
doesn't really address the original issue. To summarize, I believe the |
| 28 |
conversation went like this: |
| 29 |
|
| 30 |
user> I submitted a new ebuild with some changes, but the developer |
| 31 |
simply copied the old ebuild instead of using my new one. |
| 32 |
|
| 33 |
dev> We have problems with users simply copying old builds to create |
| 34 |
the new one without submitting a Changelog. This makes it hard |
| 35 |
for us to figure out what's been changed. |
| 36 |
|
| 37 |
I fail to see how this justifies using the old build instead of the new |
| 38 |
one. I fail, in fact, to see how the reply is even related to initial |
| 39 |
complaint.. If it's a problem for users to do "cp old.build new.build", |
| 40 |
why is it okay for a dev to do it _when a new build has been supplied_? |
| 41 |
|
| 42 |
And if there's not sufficient information to determine what's changed |
| 43 |
in the ebuild (and you don't have the time to review a diff), wouldn't |
| 44 |
it be better to kick it back to the submitter rather than ignoring what |
| 45 |
they've submitted and using the old build for the new version? A diff |
| 46 |
-q doesn't take any longer than a cp, and submitting changes without |
| 47 |
submitting a changelog seems ample justification for refusing to commit |
| 48 |
them. |
| 49 |
|
| 50 |
|
| 51 |
|
| 52 |
-- |
| 53 |
gentoo-dev@g.o mailing list |
Archives