cluster-devel.redhat.com archive mirror
 help / color / mirror / Atom feed
From: Fabio M. Di Nitto <fdinitto@redhat.com>
To: cluster-devel.redhat.com
Subject: [Cluster-devel] gfs2-utils: master - GFS2: Add script to create release tarball
Date: Mon, 11 Oct 2010 17:42:59 +0200	[thread overview]
Message-ID: <4CB33083.5020801@redhat.com> (raw)
In-Reply-To: <1286786645.2753.3.camel@dolmen>

On 10/11/2010 10:44 AM, Steven Whitehouse wrote:
> Hi,
> 
> On Sat, 2010-10-09 at 07:04 +0200, Fabio M. Di Nitto wrote:
>> did you try to use "make dist" directly from the build tree?
>>
> No, and tbh I don't really want to rely on that. My little script is
> simple and does the job just fine.

There are a few steps in there that your script will miss.

In order to run "make dist" you need to ./autogen.sh and ./configure.

Those first 2 steps will make sure that the final tarball will get a
working copy of ./configure from your system and guaranteed to be
portable. Something you will leave up to the end user to do. Different
versions of autotools/automake/autoconf can generate a different
./configure. Even a missing m4 macro could cause you headache across
distributions.

Leave alone the plans you might have to drop autotools, in the meantime
you should get advantages out of it.

make dist-check, make dist will provide you with tests over the tree and
with a properly versioned tarball.

I also suggest you look at coreutils.git Makefile.am and configure.ac to
extend configure.ac to automatically get the version from git rather
than hard encoding it in configure.ac itself (I was hinted to it today,
so I didn?t have time to look at it properly nor implement it anywhere).
The basic concept is that given an annoted tag, the make dist can create
the correct tarball (versioning and alike), including the possibility to
have version (eg.) 1.0.N-sha1 from HEAD of the tree.
Where N is the commit number after 1.0 tag and sha1 the short version of
the sha related to that release/tarball.

Fabio



      reply	other threads:[~2010-10-11 15:42 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <20101008181121.E0798120296@lists.fedorahosted.org>
2010-10-09  5:04 ` [Cluster-devel] gfs2-utils: master - GFS2: Add script to create release tarball Fabio M. Di Nitto
2010-10-11  8:44   ` Steven Whitehouse
2010-10-11 15:42     ` Fabio M. Di Nitto [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4CB33083.5020801@redhat.com \
    --to=fdinitto@redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).