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

Support rebasing forked repo against default branch in upstream repo #17

Open
dhood opened this issue Dec 9, 2016 · 4 comments
Open
Labels
enhancement New feature or request

Comments

@dhood
Copy link
Member

dhood commented Dec 9, 2016

If we specify a particular branch on the ros2 repos to test for the CI jobs, we will rebase that branch against master.

There are cases when it would be useful to do the same when testing a given ros2.repos file, such as checking that PRs from external contributors will work correctly even after being rebased.

(There are probably also cases when we would not want to rebase)

@dhood dhood added the enhancement New feature or request label Dec 9, 2016
@dirk-thomas
Copy link
Member

The rebase always happens against the specified branch in the repos file. In the case where the repos file specifies a forked repo there is simply no knowledge about the upstream repo. So I don't see how this would be possible.

@dhood
Copy link
Member Author

dhood commented Dec 9, 2016

Couldn't we rebase the repo+branch in the custom ros2.repos (e.g. dhood/* + feature) against the repo+branch in the default ros2.repos (i.e. ros2/* + master)?

That is, mimic the process that happens when we accept these PRs.

@dirk-thomas
Copy link
Member

dirk-thomas commented Dec 9, 2016

Currently there is only a single repo file - not two. So yes, in theory that is possible. But that will require a significant effort in implementing the merging of two separate repo files (with all the corner cases, e.g. some repo might only be in one of the two files, what is the correct way of merging them?).

@dirk-thomas dirk-thomas changed the title Add option for rebasing repos_url repos against master Support rebasing forked repo against default branch in upstream repo Dec 9, 2016
@dhood
Copy link
Member Author

dhood commented Dec 9, 2016

yep, good point about the edge cases. I am thinking about this for an improved process for testing external contributors' PRs (so more likely a 1:1 mapping). But perhaps one day PR testing will not require someone generating a custom repos file in the first place, so I think the more generic name for this ticket is good.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants