[OS-BUILD PATCHv2 6/6] Swap how ark-latest is built

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

 



From: Don Zickus <dzickus@xxxxxxxxxx>

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 <dzickus@xxxxxxxxxx>
---
 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 -- kernel@xxxxxxxxxxxxxxxxxxxxxxx
To unsubscribe send an email to kernel-leave@xxxxxxxxxxxxxxxxxxxxxxx
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/kernel@xxxxxxxxxxxxxxxxxxxxxxx




[Index of Archives]     [Fedora General Discussion]     [Older Fedora Users Archive]     [Fedora Advisory Board]     [Fedora Security]     [Fedora Devel Java]     [Fedora Legacy]     [Fedora Desktop]     [ATA RAID]     [Fedora Marketing]     [Fedora Mentors]     [Fedora Package Announce]     [Fedora Package Review]     [Fedora Music]     [Fedora Packaging]     [Centos]     [Fedora SELinux]     [Coolkey]     [Yum Users]     [Tux]     [Yosemite News]     [KDE Users]     [Fedora Art]     [Fedora Docs]     [USB]     [Asterisk PBX]

  Powered by Linux