← Back to team overview

hipl-core team mailing list archive

Re: [Branch ~rene-hummen/hipl/ipsec_esp] Rev 4919: ...and the missing header addition

 

On Sep 7, 2010, at 8:33 PM, Diego Biurrun wrote:
> On Mon, Sep 06, 2010 at 05:10:41PM +0000, noreply@xxxxxxxxxxxxx wrote:
>> message:
>>  ...and the missing header addition
> 
> Note that Bazaar support uncommit quite nicely, so I think it's better
> to use uncommit to fix such commits.  You can just uncommit and then
> recommit a fixed version instead of adding a new revision to fix the
> previous one.  In this way, you avoid having a broken and non-compiling
> revision in between.

Good point, although I don't like altering the history too much. It will always lead to confusion in case someone checked out the uncommitted commit in between. Nevertheless, I think for a project of our size and especially on an individual branch, this "history issue" should be of no concern.

Cheers,
René




--
Dipl.-Inform. Rene Hummen, Ph.D. Student
Distributed Systems Group
RWTH Aachen University, Germany
tel: +49 241 80 20772
web: http://ds.rwth-aachen.de/members/hummen




Follow ups

References