Skip to content

Conversation

@jerpelea
Copy link
Contributor

Reverts #179

@xiaoxiang781216
Copy link
Contributor

How about let @yamt provide the new PR instead reverting the whole patchset?

@jerpelea jerpelea closed this Jan 29, 2020
@jerpelea jerpelea reopened this Jan 29, 2020
@patacongo
Copy link
Contributor

This should be closed, right? It was merged prematurely. For complex changes.. ESPECIALLY changes involving the build system, we must wait for comments. Committing build system change blindly is almost always fatal.

We are waiting for the updates from the author now, right? So this should be closed.

@jerpelea
Copy link
Contributor Author

I am sorry for accidentally merging this one

@jerpelea jerpelea closed this Jan 29, 2020
@jerpelea jerpelea reopened this Jan 29, 2020
@jerpelea
Copy link
Contributor Author

we can revert it and wait to get the fixes or we can merge my revert and wait to a new PR

@patacongo
Copy link
Contributor

Don't worry about it. We appreciate all of your help. We all sometimes make the decision that we would later do differently. I think I have done many like that and I usually hear about it too.

I think it would be simpler to just add the fixes rather than reverting and reapplying the whole thing, But we need make sure that everyone understands how we are going to proceed and make sure that fixes will actually happen.

@patacongo
Copy link
Contributor

"we can revert it and wait to get the fixes or we can merge my revert and wait to a new PR"-- less turmoil if we can wait. I propose that if nothing happens in 24 hours, we go ahead and revert. I can do the merge for you then.

@jerpelea jerpelea closed this Jan 30, 2020
@Ouss4
Copy link
Member

Ouss4 commented Jan 30, 2020

@jerpelea Should the revert-179-osx-sim branch be deleted as well?

@jerpelea jerpelea deleted the revert-179-osx-sim branch January 30, 2020 10:10
@jerpelea
Copy link
Contributor Author

done !
thanks for reminding me

@TA1DB
Copy link

TA1DB commented Oct 25, 2020

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.

5 participants