[GH #877] Remove Parrot::Revision #945

Merged
merged 5 commits into from Mar 22, 2013

Conversation

Projects
None yet
3 participants
Contributor

paultcochrane commented Mar 22, 2013

This set of changes removes Parrot::Revision as mentioned in GH #877. Comments welcome.

Paul Cochrane added some commits Mar 22, 2013

Paul Cochrane
[GH #877] Removed @revision@ text in myconfig output
The @revision@ information was only useful back in the Subversion days and
consequently (since we are now using Git) isn't necessary anymore.
Paul Cochrane
[GH #877] Removed auto::revision from configure steps
We wish to remove the Parrot::Revision functionality, so this dependence
within the configure step needs to be removed.
Paul Cochrane
[GH #877] Removed Parrot::Revision dependence in Configure.pl
Since we wish to remove Parrot::Revision, we therefore no longer need to
update the Parrot revision number when configuring.
Paul Cochrane
[GH #877] Removed config/auto/revision.pm and its tests
In issue GH#877 the old Parrot::Revision code should be removed.  This
removes the configure step and its associated tests.

The references in t/configure/testlib/*defectivefoobar have also been
removed.
Paul Cochrane
[GH #877] Removed Parrot::Revision
Now that the dependencies on Parrot::Revision have been removed, it's now
possible to remove Parrot::Revision itself.  The tests pass, so all is good.
Owner

leto commented Mar 22, 2013

Looks great, merging.

leto added a commit that referenced this pull request Mar 22, 2013

@leto leto merged commit 58e5de9 into parrot:master Mar 22, 2013

Contributor

jkeenan commented Mar 22, 2013

On 3/22/13 1:01 PM, paultcochrane wrote:

This set of changes removes Parrot::Revision as mentioned in GH #877
#877. Comments welcome.

All hail Paul Cochrane: King of the Cage Cleaners!

Good to hear from you again, Paul!

Jim Keenan

Contributor

paultcochrane commented Mar 23, 2013

Hi Jim,
thanks! It's nice to be back and able to help out again :-)
Paul

@paultcochrane paultcochrane deleted the paultcochrane:ptc/remove_parrot_revision branch Mar 7, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment