[LLVMdev] [PATCH] llvm-config: add svn revision to --version

David Greene
Mon, 18 May 2009 21:17:10 -0000

On Monday 18 May 2009 13:35, Frits van Bommel wrote:
> Török Edwin wrote:
> > I think you don't need to support all trunk revisions.
> > People using trunk could be told to either update to a certain
revision,
> > or stick to a released version.
>
> We don't support /all/ trunk revisions, but we don't like to force
upgrades
> unless it's really needed so we try to at least support most recent
> revisions.
>
> One of the reasons is that not all of our developers use the same trunk
> revision, and recompiling LLVM trunk is not something everyone likes to do
> once a day. (It's nice if you can put it off for a couple of days
without
> things breaking)

There's an easy way to fix these kinds of problems.  Deprecate interfaces
for
one release.  I brought this up before but the idea was quickly shot down.
It's insane to litter your code with #ifdefs based on SVN revisions.

This is exactly the kind of situation that the LLVM community needs to
address.  Changing APIs is fine.  Ripping out old versions without first
deprecating them for a release causes nightmares all around.

                           -Dave