Re: XPSSourceInformation.java

[Strange, this mail didn't get to the mailing list.]


Hi Christian,

Christian Egli wrote:
> I just spent a few good hours of my life wondering why my publication
> did not work anymore. Turns out that I had to revert a change in
> XPSSourceInformation.java to make it work again.
> 
> There's a few things that I'd like to say about this change:
> 
> 1. I don't particularly like <change author="foobar"> tag. This kind
>    of information is stored in CVS. In the source code it's just
>    noise.

You're absolutely right. I just added these tags to show Michael
where my changes are when I sent him the file. That was a special
situation (communicating files without CVS) and we don't have to
discuss it, do we? ;-)


> 2. I suppose if we had some kind of regression test this wouldn't
>    have happened.

+1

I'm looking forward to Gregor's jUnit introduction and to seeing
us all write unit tests :)


> 3. For people with productive servers that sync with us by CVS this
>    can be a nasty surprise. We should probably separate development
>    like that on a development branch and keep changes of the stable
>    branch to a minimum.

Maybe we soon arrive at a state where we can release a version as
"stable"? I'm not fond of having two parallel code versions at all.

I guess I have to add the XPSSourceInformation to my task list
on Thursday and Friday. Otherwise, I will always have to patch my
version as long as I'm using Windows.


Andreas