[PATCH libdrm] Simplify the RELEASING steps based on current release.sh.
Emil Velikov
emil.l.velikov at gmail.com
Wed Jul 20 23:11:30 UTC 2016
On 20 July 2016 at 20:27, Eric Anholt <eric at anholt.net> wrote:
> Since release.sh creates and pushes a libdrm-$VERSION tag for us,
> there's no need to also have the user manually generating a $VERSION
> tag as well.
>
> I also dropped the "optional" part of distcheck. You shouldn't have
> pushed master with a version bump that hasn't passed distcheck.
> ---
> RELEASING | 26 ++++++--------------------
> 1 file changed, 6 insertions(+), 20 deletions(-)
>
> diff --git a/RELEASING b/RELEASING
> index 62c5be9fafe9..262ca08d26c4 100644
> --- a/RELEASING
> +++ b/RELEASING
> @@ -9,21 +9,14 @@ However, this is up to whoever is driving the feature in question.
>
> Follow these steps to release a new version of libdrm:
>
> - 1) Ensure that there are no local, uncommitted/unpushed
> - modifications. You're probably in a good state if both "git diff
> - HEAD" and "git log master..origin/master" give no output.
> -
Nice. release.sh already checks this for us - no point in listing it
here afaict.
> - 2) Bump the version number in configure.ac. We seem to have settled
> + 1) Bump the version number in configure.ac. We seem to have settled
> for 2.4.x as the versioning scheme for libdrm, so just bump the
> micro version.
>
> - 3) Run autoconf and then re-run ./configure so the build system
> + 2) Run autoconf and then re-run ./configure so the build system
> picks up the new version number.
>
I have sent some patches [1] for release.sh that make this step
obsolete. If you can weight-in on the topic that'll be appreciated.
[1] https://patchwork.freedesktop.org/series/9382/
Patch 9/10 in particular
> - 4) (optional step, release.sh will make distcheck for you, but it can be
> - heart warming to verify that make distcheck passes)
> -
> - Verify that the code passes "make distcheck". Running "make
> + 3) Verify that the code passes "make distcheck". Running "make
> distcheck" should result in no warnings or errors and end with a
> message of the form:
>
Note: There's still a few warnings which we should squash one of these days.
> @@ -36,20 +29,13 @@ Follow these steps to release a new version of libdrm:
> Make sure that the version number reported by distcheck and in
> the tarball names matches the number you bumped to in configure.ac.
>
> - 5) Commit the configure.ac change and make an annotated tag for that
> - commit with the version number of the release as the name and a
> - message of "libdrm X.Y.Z". For example, for the 2.4.16 release
> - the command is:
> -
> - git tag -a 2.4.16 -m "libdrm 2.4.16"
> -
Yes please. The duplicate [but not quite] tag have always confused me.
Fwiw:
Reviewed-by: Emil Velikov <emil.l.velikov at gmail.com>
-Emil
More information about the dri-devel
mailing list