From: Don Zickus <[email protected]>

Today ark-latest is built by starting with ark-patches, merging
os-build and applying 'extra' patches from gitlab, and finally
tagged.

In preparation for a single tree workflow where ark-patches disappears,
lets swap the process to make it easier for ark-patches to be
removed.

Instead let's build ark-latest by starting with os-build, merge
ark-patches, apply 'extra' patches and tag  it.

Later, it becomes easy to drop the ark-patches merge and when no
'extra' patches exist, ark-latest and os-build are identical.

However, this leads to a changelog quirk.  The genspec logic
thinks all the ark-patches are new additions and adds entries
for that whole branch into the changelog because the MARKER has
not changed.

This is noise.  For now I created a temporary hack to work around
this until a single workflow is implemented and the problem naturally
goes away.

Signed-off-by: Don Zickus <[email protected]>
---
 redhat/genspec.sh                       | 8 ++++++++
 redhat/scripts/ci/ark-create-release.sh | 4 ++--
 2 files changed, 10 insertions(+), 2 deletions(-)

diff --git a/redhat/genspec.sh b/redhat/genspec.sh
index b72dffe14729..fbe275f6f4e7 100755
--- a/redhat/genspec.sh
+++ b/redhat/genspec.sh
@@ -195,6 +195,14 @@ if [ "$MARKER" != "$LAST_MARKER" ]; then
        rm $clogf.tmp $clogf.unique
 fi
 
+# HACK temporary hack until single tree workflow
+# Don't reprint all the ark-patches again.
+if [ -n "$(git log --oneline --first-parent --grep="Merge ark patches" 
${lasttag}..)" ]; then
+       # Throw away the clogf and just print the summary merge
+       echo "" > $clogf
+       echo "- Merge ark-patches" >> $clogf
+fi
+
 LENGTH=$(wc -l $clogf | awk '{print $1}')
 
 #the changelog was created in reverse order
diff --git a/redhat/scripts/ci/ark-create-release.sh 
b/redhat/scripts/ci/ark-create-release.sh
index 3181f32a56e8..0d4cfd9181e5 100755
--- a/redhat/scripts/ci/ark-create-release.sh
+++ b/redhat/scripts/ci/ark-create-release.sh
@@ -43,10 +43,10 @@ if git tag -v "$UPSTREAM_REF" > /dev/null 2>&1; then
        RELEASE_BRANCHES=" ark/$UPSTREAM_REF ark/patches/$UPSTREAM_REF"
 else
        # This is a snapshot release so we're only going to make a tag
-       git checkout --detach ark-patches && git describe
+       git checkout --detach os-build && git describe
        RELEASE_BRANCHES=""
 fi
-git merge -m "Merge configuration and build scripts" os-build
+git merge -m "Merge ark patches" ark-patches
 
 MR_PATCHES=$(gitlab project-merge-request list --project-id=${PROJECT_ID} \
        --labels="Include in Releases" --state=opened | grep -v "^$" | sort | \
-- 
GitLab
_______________________________________________
kernel mailing list -- [email protected]
To unsubscribe send an email to [email protected]
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/[email protected]

Reply via email to