Skip to content

Conversation

@cuviper
Copy link
Contributor

@cuviper cuviper commented Sep 14, 2018

No description provided.

@glaubitz
Copy link
Contributor

Can we rather merge #609 which adds sparc and sparc64 as well?

@dhardy
Copy link
Member

dhardy commented Sep 15, 2018

How come two PRs pop up adding the same functionality on the same day?

I prefer the style here: more code reuse.

@glaubitz
Copy link
Contributor

We came from the same thread and had the same idea, accidents happen. However, @cuviper forgot adding sparc and sparc64. My PR adds those and is more consistent.

@dhardy
Copy link
Member

dhardy commented Sep 17, 2018

Okay, #609 merged. Thank you both.

@dhardy dhardy closed this Sep 17, 2018
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.

3 participants