Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Use DEP-14 branch names debian/latest and upstream/latest, and upstream vcs remote name 'upstreamvcs' #247

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Use DEP-14 branch names debian/latest and upstream/latest, and upstream vcs remote name 'upstreamvcs' #247

wants to merge 2 commits into from

Conversation

ottok
Copy link
Contributor

@ottok ottok commented Feb 19, 2025

See commits for details. This is a re-submission of #225, pending to be merged potentially in the summer of 2025.

ottok added 2 commits February 18, 2025 15:35
@ottok
Use DEP-14 branch names debian/latest and upstream/latest
d5fd544 
In DEP-14, the preferred branch name for the Debian packaging target
branch is `debian/latest` and the preferred name for the upstream import
target branch is `upstream/latest`.

Note that the upstream development branch name can be whatever and
should stay as it is upstream, typically `main` or `master`. The branch
`upstream/latest` should not point to the latest upstream development
commit, but to the latest commit that was used as the upstream release
that the Debian revision was derived from.
@ottok
Always call upstream git remote upstreamvcs
8710283 
Instead of using various different upstream remote names, use the one
and same upstream git remote name consistently. As the name pick
`upstreamvcs` just as git-buildpackage does, so that if anybody runs
`gbp clone` they will automatically end up with the same git remotes and
branches as anyone in to go-team.
NightTsarina
NightTsarina requested changes Feb 19, 2025
View reviewed changes
Copy link
Contributor

@NightTsarina NightTsarina left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Most of the changes in this PR are not very controversial and I support them, even if I personally prefer upstream/<upstream HEAD> instead of upstream/latest (which is valid DEP-14 as we are not importing tarballs but following git history.

But big NACK to the change of the debian branch, you still haven't given a reason to change, except a misinterpretation of DEP-14. This PR cannot be merged until there is a consensus on this topic.

@umlaeute
Copy link

except a misinterpretation of DEP-14

how's debian/latest a misinterpretation?
i fully acknowledge that DEP-14 allows different names, and that there is controversy about which name to pick and that otto is pushing rather this controversial name strongly.
but nevertheless is debian/latest is explicitly suggested (among others) by DEP-14, so it's a valid interpretation.

@guillemj
Copy link
Contributor

What is a misinterpretation is stating the debian/latest is the preferred branch name by the DEP, which is not true.

@ottok
Copy link
Contributor Author

ottok commented Feb 21, 2025

I did not invent debian/latest or upstream/latest, they are from DEP-14. I have however been using them successfully for many years and I think they make sense. Personally I would feel odd if I was using a branch called debian/unstable as the permanent default branch in git and the development branch, and occasionally uploading to experimental from a branch called 'unstable'. The reason DEP-14 (which I didn't write) recommends debian/latest is presumably to avoid mixing up default git branch name with the target archive name.

@NightTsarina
Copy link
Contributor

I did not invent debian/latest or upstream/latest, they are from DEP-14. I have however been using them successfully for many years and I think they make sense. Personally I would feel odd if I was using a branch called debian/unstable as the permanent default branch in git and the development branch, and occasionally uploading to experimental from a branch called 'unstable'. The reason DEP-14 (which I didn't write) recommends debian/latest is presumably to avoid mixing up default git branch name with the target archive name.

@ottok Here lies the problem:

  1. DEP-14 does not mandate debian/latest, debian/<suite> is also a valid option.
  2. The team does not use debian/unstable for uploads to experimental, but debian/experimental.
  3. Despite your experience, the go-team has been using DEP-14 successfully for many years, and there is absolutely no benefit in this change, but a huge cost.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@NightTsarina NightTsarina NightTsarina requested changes

Requested changes must be addressed to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@ottok @umlaeute @guillemj @NightTsarina