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.

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

Partial merge of rb 2.5 #988

Closed
wants to merge 6 commits into from
Closed

Partial merge of rb 2.5 #988

wants to merge 6 commits into from

Conversation

devernay
Copy link
Member

@devernay devernay commented Jul 22, 2024

There are conflicts when merging RB-2.5into RB-2.6, so I could not do a full merge.
This is just a cherrypick of some of the 18 commits, but those related to:

  1. github actions
  2. cmake
  3. windows packages

were not merged. Here's the full list of missing commits:

@acolwell or @rodlie could you check what needs to be merged and do a full merge of RB-2.5 into RB-2.6? We should always merge changes from RB-2.x into RB-2.y if x < y, so that RB-2.y is fully ahead of RB-2.x. Hopefully this will not be an issue anymore now that RB-2.6 is the default branch.

mruegenberg and others added 6 commits July 22, 2024 16:25
Bountysource appears to be dead, so it should be removed
* RotoPaint: always draw the overlay when the tool is part of the overlay
@devernay devernay marked this pull request as ready for review July 22, 2024 14:38
@devernay devernay requested review from acolwell and rodlie July 22, 2024 14:39
@acolwell
Copy link
Collaborator

There are conflicts when merging RB-2.5into RB-2.6, so I could not do a full merge. This is just a cherrypick of some of the 18 commits, but those related to:

  1. github actions
  2. cmake
  3. windows packages

were not merged. Here's the full list of missing commits:

@acolwell or @rodlie could you check what needs to be merged and do a full merge of RB-2.5 into RB-2.6? We should always merge changes from RB-2.x into RB-2.y if x < y, so that RB-2.y is fully ahead of RB-2.x. Hopefully this will not be an issue anymore now that RB-2.6 is the default branch.

I merged everything from 2.5 to 2.6 before I requested 2.6 become the default branch. (#977) I believe your MacPorts change and RotoPaint fix are the only things that actually need to be merged.

@rodlie
Copy link
Contributor

rodlie commented Jul 22, 2024

Just resolve the merge issues?

Commit 73e7dd8 breaks against RB-2.5, just ignore the changes coming from RB-2.5.

Also the same with the macports changes in 4253886, but instead accept the changes from RB-2.5.

@rodlie
Copy link
Contributor

rodlie commented Jul 22, 2024

@acolwell
Copy link
Collaborator

acolwell commented Jul 22, 2024

This should work? https://github.com/NatronGitHub/Natron/compare/RB-2.6...rodlie:Natron:merge-RB256?expand=1

@rodlie that looks good to me. I think I accidentally didn't do a 'merge commit' when I merged everything before so that is why we have all the commits that were merged already still showing up in this merge. I think what you have there will be a better option than this cherry pick PR because I think it will make future merges from 2.5(please no more) less painful.

Sorry for causing this issue.

@devernay
Copy link
Member Author

cancelling

@devernay devernay closed this Jul 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants