SF.net SVN: avogadro: trunk

Development
1

On Sep 6, 2007, at 12:20 PM, benoitjacob@users.sourceforge.net wrote:

Let CMake produce a special header file definining a string
SVN_REVISION_STRING.
The header is installed at $CMAKE_INSTALL_PREFIX/include/avogadro/
svn.h
This allows applications using libavogadro to be aware of which
revision they are using.

This is a great idea, since it makes the About box more useful –
both the “install version” (e.g., 0.2) and the SVN build revision.

But when I tried it, on running cmake, it immediately tried to
install the header:

CMake Error: Could not open file for write in copy operation /usr/
local/include/avogadro/svn.h.tmp
CMake Error: : System Error: Permission denied

Shouldn’t we create a temporary svn.h in the libavogadro src/build
directories and install later? I think most people will be surprised
that they need to run cmake as root.

Cheers,
-Geoff

2

On Thursday 06 September 2007 18:24:30 Geoffrey Hutchison wrote:

But when I tried it, on running cmake, it immediately tried to
install the header:

CMake Error: Could not open file for write in copy operation /usr/
local/include/avogadro/svn.h.tmp
CMake Error: : System Error: Permission denied

Yeah, Simon had the same problem. svn up and retry now :slight_smile:

Shouldn’t we create a temporary svn.h in the libavogadro src/build

exactly what i did in the second commit :slight_smile:

directories and install later?

currently it’s no longer installed (since the second commit). Feel free to
make svn.h installed if you think this can be useful to more apps than just
avogadro.

Benoit