[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

Re: [virt-tools-list] [PATCH 1/3] delete autobuild.sh

On 04/03/2013 04:21 PM, Daniel P. Berrange wrote:
On Wed, Apr 03, 2013 at 04:07:34PM -0400, Gene Czarcinski wrote:
On 04/03/2013 03:01 PM, Daniel P. Berrange wrote:
On Wed, Apr 03, 2013 at 02:12:32PM -0400, Gene Czarcinski wrote:
There may still be an autobuild server and it may be
necessary for vir--manager rpms to be built by it, but
this script depends on autotools and will not work
with the current distutils approach.
Signed-off-by: Gene Czarcinski <gene czarc net>
NACK, fix the script to work with distutils, don't delete it.

OK, I am not familiar with this server.  Could you provide me with
more info or point me to where I can see the environment this script
must run in?
The intent is that you merely need do

    export AUTOBUILD_INSTALL_ROOT=$HOME/tmp/autobuld

and it will do a configure, build, install putting all the files into
$HOME/tmp/autobuild as the install prefix. Then create a release dist
and run the RPM buld.

Also, should this be a shell script or python?
Shell, as now. Just need to replace the calls to 'configure' and
'make' etc with corresponding distutils shell commands. eg
replace "make install" with "python setup.py install"

That was easy ... got the dist and rpms done, will do the install in a bit.

The problem (there always is one) is another of the patches I submitted ... the one changing the way virt-manager version is done. You have a choice of using a "fixed" version or one based on the move recent tag (the first part of git-describe). Regardless, if the tag of the git-describe is not pointing to the current commit or if the base version has version-<something>, then ".gityyyymmdd" is appended to the external-version-id which is used in the tarball and rpm names. Thus, the result of autobuild.sh are some rpm files with filenames of more than sixty bytes long.

Any suggestions? I would like to retain the "snapshot" id part of my version patch but when mixed with "extra release", it is way too long.

BTW, one solution is to add some "special" commands to setup.py to just handle the needs of autobuild. What would you like the "unique name" to look like?


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]