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

Fix merge problem #2548

Merged
merged 1 commit into from Feb 19, 2014
Merged

Fix merge problem #2548

merged 1 commit into from Feb 19, 2014

Conversation

wddgit
Copy link
Contributor

@wddgit wddgit commented Feb 19, 2014

Fix problem when 7_1_X was merged into 7_1_THREADED_X.
There was a conflict and some necessary changes were
not merged. This is causing compilation to fail in
7_1_THREADED_X.

Fix problem when 7_1_X was merged into 7_1_THREADED_X.
There was a conflict and some necessary changes were
not merged. This is causing compilation to fail in
7_1_THREADED_X.
@Dr15Jones
Copy link
Contributor

@ktf Giulio, please bypass signatures since this was a merge failure problem.

ktf added a commit that referenced this pull request Feb 19, 2014
@ktf ktf merged commit f0355b1 into cms-sw:CMSSW_7_1_THREADED_X Feb 19, 2014
@ktf
Copy link
Contributor

ktf commented Feb 19, 2014

Do we want to revisit the merge strategy?

@wddgit
Copy link
Contributor Author

wddgit commented Feb 20, 2014

Today I checked all the files modified in the random number service commit that I put on the threaded branch. The good news is that there are no more merge conflict problems related to that commit. On the other hand, future commits to 7_1_X could still conflict with that random number service branch commit. So far there have been 4 merge problems related to it that have been fixed, 3 by me and one by Andrea Bocci. At least they all showed up in obvious ways, compilation failures or unit test failures. I did not check anything other than that one commit and the related fixes.

@wddgit wddgit deleted the fixMergeProblem branch April 25, 2014 20:19
ggovi pushed a commit to ggovi/cmssw that referenced this pull request Jan 11, 2017
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.

None yet

3 participants