In some places we also reference main which will be a symlink to the rawhide branch and we currently still reference master to be backward compatible (even though once the rawhide branch appears the master branch will be removed and blocked).
Signed-off-by: Pierre-Yves Chibon pingou@pingoured.fr
This is based on a quick git grep and quite un-tested. The test suite fails for me in tox as well as when running nosetests manually :(
rebased onto fb50218267327bbd09b4692fa17b7cf824e89d19
rebased onto 885043c9d2cb883f03a747c6d9a3c858a0d8864c
1 new commit added
More updates to 'rawhide' default branch
I added more changes, but this also needs changes to tests.
These changes to tests are sufficient to make them pass locally for me. https://pagure.io/fork/lsedlar/fedpkg/c/6f9b0667800b8fad5fcec2afde2617f8c8e9d0fe?branch=fix-review-424
Is the tooling that handles new branches updated to handle requests for rawhide branches? This patch would change the JSON that is submitted to list rawhide instead of master.
rawhide
master
I've enabled PRs on my fork so you can open a PR against my fork ;-)
It's on the todo @humaton and @mohanboddu should be looking at this iirc
Update tests for rawhide branch rename
considering the state of these two projects, I doubt we'll change their default branches
FTR I didn't mean the default branch of the request repo itself, but the content of submitted requests: https://pagure.io/releng/fedora-scm-requests/issue/31722 With this patch, branch would switch from master to rawhide, which is probably a wanted change, and I just want to make sure tools processing the ticket can handle it.
branch
pretty please pagure-ci rebuild
Isn't this should be rawhide as well and while we are at it, better change the test name as well.
Maybe keep this one as is as it shows the backward compatibility and add a new one?
Sounds good to me.
Adding test_expand_rawhide test
4 new commits added
Adjust fedpkg for the new default branch in dist-git: rawhide
Looks all good to me! :)
Code looks correct. Tests are also passing. Don't know why they were not exected automatically, but I run Jenkins manually. I took a patch to build fedpkg scratch build. I tried at least building functionality from dist-git branch which has old master branch. It worked. My only concern is whether fedora-scm (already) creates rawhide branches for fedpkg requests. Will they create "symlink" branches or just a single one? There are 4 commits in the PR. They look complicated because at least first two commits are interfering together. The best approach for me would be 2 commits for functionality and unittests. If you don't mind, I can try to transform it during the merge. I would like to preserve authors (in case of 2 authors, I would name one of you in the commit message).
OK, answering to myself :) https://fedoraproject.org/wiki/Changes/GitRepos-master-to-main: Along with the src.fedoraproject.org changes, we will update the branch filter list and forbid 'master' from being pushed at all. Additionally, we will establish a sym-ref between 'rawhide' and 'main' as a convenience for people who prefer to use 'main' branch for rawhide.
My only concern is whether fedora-scm (already) creates rawhide branches for fedpkg requests. Will they create "symlink" branches or just a single one?
@humaton @mohanboddu do you know the status of fedscm-admin?
My only concern is whether fedora-scm (already) creates rawhide branches for fedpkg requests. Will they create "symlink" branches or just a single one? @humaton @mohanboddu do you know the status of fedscm-admin?
fedscm_admin creates the main "symlink".
And can handle new package/branch requests asking for "rawhide" instead of "master"?
Commit 34a971b2 fixes this pull-request
Pull-Request has been merged by onosek
Didn't find where the fedora-scm-request execution code resides, so I hope it is working as expected. I regrouped 4 commits - resulting in 2. And I am going to create and release a patch. Thank you for the code, guys.
All fedpkg updates are listed here: https://bodhi.fedoraproject.org/users/onosek
In some places we also reference main which will be a symlink to
the rawhide branch and we currently still reference master to be
backward compatible (even though once the rawhide branch appears
the master branch will be removed and blocked).
Signed-off-by: Pierre-Yves Chibon pingou@pingoured.fr