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

[RFC] remove Parrot support #298

Merged
merged 14 commits into from Jul 26, 2016
Merged

[RFC] remove Parrot support #298

merged 14 commits into from Jul 26, 2016

Conversation

moritz
Copy link
Contributor

@moritz moritz commented Jul 24, 2016

Activity levels (or the absence thereof) in Parrot in the last half a year to year indicates to me that we should give up on the idea of resurrecting the Rakudo Parrot backend, and as such I see no reason to keep the NQP backend.

This simplifies the build system, decreases the checkout size on disk, and gives fewer false positives when search for relevant things in the source code.

If no major objections from NQP or Rakudo core contributors arise, I plan to merge this pull request in about a week. Maybe earlier if the overall resonance is positive.

CC to @jnthn and @pmichaud just in case :)

@arnsholt
Copy link
Contributor

No objections on my part.

@lizmat
Copy link
Contributor

lizmat commented Jul 24, 2016

nor from me

@pmichaud
Copy link
Contributor

I have no objections to removing the parrot components from nqp's master branch. Perhaps create a "parrot" branch or tag to make it easy to get to the parrot-specific stuff if someone decides to try to restore it.

Pm

@coke
Copy link
Contributor

coke commented Jul 25, 2016

After some initial negative thoughts about this, I'm on board; especially if we tag the last known commit with a parrot build on it.

@coke
Copy link
Contributor

coke commented Jul 25, 2016

There are still many references to parrot here. Mind if I add a few more removals, @moritz ?

@moritz
Copy link
Contributor Author

moritz commented Jul 25, 2016

@coke I have no objections to a more thorough removal. Consider this a collaborative effort.

@moritz
Copy link
Contributor Author

moritz commented Jul 25, 2016

@pmichaud ok, I'll create a branch before merging this PR.

@jnthn
Copy link
Contributor

jnthn commented Jul 25, 2016

I'm also fine with this. So far as I'm aware, NQP on Parrot has not built for a number of months, and I can't recall anybody complaining about that - which is probably indicative of the interest level.

Leaving a branch behind is OK, though IMHO a tag would be more natural (tags are for marking significant points in development, whereas branches are more for ongoing development). If anybody fancied the leg work, putting that tag on the last release that actually built against Parrot would maybe be most fitting, but I sure ain't volunteering anyone for that task. :-) A tag on the last commit before we remove it work be fine by me.

@moritz moritz merged commit 5a717bb into master Jul 26, 2016
@moritz moritz deleted the noparrot branch July 26, 2016 11:51
@moritz
Copy link
Contributor Author

moritz commented Jul 26, 2016

Thanks for the comments everybody, I've merged this now, and created a parrot tag, which is the last commit before the parrot removal.

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

6 participants