Here is the rationale for tagging all repositories:
- In Replicant 6.0 we now have mirrored all the repositories we use, so we
  can now tag them all.
- For libsamsung-ipc, as the tag as replicant-<version> in its name, it
  cannot conflict with future tags for other distributions, or future
  Replicant versions.
- Even if libsamsung-ipc is meant to be shared across different
  distributions, having a record of which Replicant versions use
  which libsamsung-ipc versions in libsamsung-ipc is very important as
  libsamsung-ipc is heavily tested with Replicant and users reporting bugs
  in Replicant typically know the Replicant version not the exact
  libsamsung-ipc commit used.

In addition, we need to tag the manifest otherwise we have an error like
that:
    $ ./release.sh replicant-6.0 replicant-6.0-0004-rc2 all
    Releasing metadata
    fatal: ambiguous argument 'replicant-6.0-0004-rc2':
    unknown revision or path not in the working tree.
    Use '--' to separate paths from revisions, like this:
    'git <command> [<revision>...] -- [<file>...]'
And it is better to have it done automatically, in the same way than for
the other repositories.

To do that, we will document the release procedure in a subsequent commit
to make sure that no step is missed.

It will make sure that .repo/manifests has the required tag before being
used by release.sh by:
- Making sure that the people doing the release don't forget to add patches
  in the manifest/ directory. Patches are needed to change the revision
  to the release tag.
- Making sure that the tag pushed by releasetag.sh.
- Making sure that people doing the release will re-download the source for
  building, so the tag will be in .repo/manifests.

Signed-off-by: Denis 'GNUtoo' Carikli <gnu...@cyberdimension.org>
---
 releasetag.sh | 9 ++-------
 1 file changed, 2 insertions(+), 7 deletions(-)

diff --git a/releasetag.sh b/releasetag.sh
index 90e2c27..a1ef8f5 100755
--- a/releasetag.sh
+++ b/releasetag.sh
@@ -33,19 +33,14 @@ set_color
 tag_repos () {
        dir=$( pwd )
        cd "$REPLICANT_DIR"
-
-       ALL_MIRROR_REPOS=$( repo forall -c ' echo "$REPO_REMOTE$REPO_PROJECT" | 
grep lineage-mirror \
-                                               | cut -c15-' )
-       ALL_REPLICANT_REPOS=$( repo forall -c ' echo 
"$REPO_REMOTE$REPO_PROJECT" | grep replicant \
-                                               | cut -c10- | grep -v 
"manifest.git" || true ' )
        TAGGING_CMD="echo \"Tagging $color_start \$REPO_PROJECT $color_end\" && 
\
                        git tag -u $RELEASE_KEY -m \"Replicant $VERSION 
$RELEASE images release\" \
                                \"replicant-$VERSION-$RELEASE\""
        PUSHING_CMD="echo \"Pushing $color_start \$REPO_PROJECT $color_end\" && 
\
                        git push \"git@$PUSH_REMOTE:\$REPO_PROJECT\" 
\"replicant-$VERSION-$RELEASE\""
 
-       repo forall $ALL_MIRROR_REPOS -c "$TAGGING_CMD && $PUSHING_CMD"
-       repo forall $ALL_REPLICANT_REPOS -c "$TAGGING_CMD && $PUSHING_CMD"
+       repo forall -c "$TAGGING_CMD && $PUSHING_CMD"
+       repo forall -c "$TAGGING_CMD && $PUSHING_CMD"
 
        cd "$dir"
 }
-- 
2.27.0

_______________________________________________
Replicant mailing list
Replicant@osuosl.org
https://lists.osuosl.org/mailman/listinfo/replicant

Reply via email to