protonscr

release-1.0 new way to track dxvk source releases?

dxvkclosed
doitsujin/dxvk#995 · opened 2019-04-02 by bobwya · updated 2019-04-11 · 12 comments · github
Bbobwya 2019-04-02 github

I see that the master branch is no longer tracking dxvk releases.
Is the release-1.0 branch the accepted way to track newer releases?

I am asking with reference to the winetricks dxvk verb.
This currently uses the RELEASE file, to obtain the current release.

Thanks

Ddoitsujin maintainer 2019-04-02 github

Hm, yeah, I can see why this is an issue. release-1.0 has diverged from master and only receives cherry-picked bug fixes. master will at some point (soon-ish) turn into 1.1.

Would it be useful if I maintained a latest-release branch that always points to the latest tagged release, and thus always contains an up-to-date RELEASE file? I'd just force-push to it whenever tagging a new one.

Please let me know if there are any more elegant ways to solve this, I just can't think of anything better right now.

Bbobwya 2019-04-02 github

KISS! So yeah, a latest-release branch would be cool.
Thanks! :-)

Aaustin987 2019-04-02 github

A latest-release branch would work, but I'm not clear on why that couldn't be done in master instead?

I have two concerns with that approach.

  1. you might forget to update the service branch
  2. if there's 1.0, 1.0.1 and 1.1, then you tag 1.0.2, what's the latest? IMO it's 1.1, but it's unclear from your comment which it would be.

Anyway, as long at there's a file we can query in plain text, we can make it work 😀

Pphush0 2019-04-03 github

Release can be done by tagging point release. Then with git it is simple.

Ddoitsujin maintainer 2019-04-03 github

@austin987 because having a commit "Release 1.0.2" in the master branch is a bit weird and confusing when the master branch itself isn't based off 1.0.2.

if there's 1.0, 1.0.1 and 1.1, then you tag 1.0.2, what's the latest? IMO it's 1.1, but it's unclear from your comment which it would be.

I don't plan to support multiple branches at the same time with new releases, however if that should ever happen, it will point the latest branch (i.e. 1.1).

@phush0 What do you mean? Releases are already tagged, but getting a list of tags requires a git checkout which should explicitly be avoided.

Ddoitsujin maintainer 2019-04-03 github
Pphush0 2019-04-03 github

Never mind, my misunderstanding of the problem.

Bbobwya 2019-04-03 github

Thanks, that's great!

Aaustin987 2019-04-03 github

@doitsujin, I agree. Do you plan to tag 1.0.2 in latest-release, after 1.1.0? That's what I'm trying to understand/avoid.

Aaustin987 2019-04-03 github

Saw edit, that works, thanks!

Ddoitsujin maintainer 2019-04-06 github

Before anyone complains: I'm not going to update that branch to 1.1, that release turned out to be a failure pretty much immediately.

Ddhewg 2019-04-11 github

You can just use the github api to get the latest release:
https://api.github.com/repos/doitsujin/dxvk/releases/latest

It's what I'm doing for the galliumnine verb already. For exactly that reason, there's no need to rely on any commited file, branch or whatever.

Nothing extracted yet.