Ticket #310 (closed todo: fixed)

Opened 13 years ago

Last modified 13 years ago

remove --pmc option to Configure.pl

Reported by: coke Owned by: jkeenan
Priority: normal Milestone:
Component: configure Version:
Severity: medium Keywords: configure pmc
Cc: Language:
Patch status: applied Platform:

Description

There's no need to allow builders of parrot to specify which PMCs to configure with.

This option, any supporting code, and tests should be removed.

Change History

  Changed 13 years ago by jkeenan

Historical note: This was alluded to in  Paul Cochrane's post to RT 43172.

in reply to: ↑ description   Changed 13 years ago by jkeenan

  • keywords configure pmc added
  • component changed from none to configure

Replying to coke:

There's no need to allow builders of parrot to specify which PMCs to configure with. This option, any supporting code, and tests should be removed.

Coke:

(1) Does this qualify as a deprecation which has to be put through the cycle?

(2) Are people actually agreed on this -- or is this more an RFC?

follow-up: ↓ 5   Changed 13 years ago by coke

(1) - I suppose so. Added in r37067

(2) - I opened a ticket to get feedback. No one has said "no" in 3 weeks, so I win (at least for now)

  Changed 13 years ago by jkeenan

  • status changed from new to assigned
  • owner set to jkeenan

in reply to: ↑ 3   Changed 13 years ago by jkeenan

Replying to coke:

(1) - I suppose so. Added in r37067 (2) - I opened a ticket to get feedback. No one has said "no" in 3 weeks, so I win (at least for now)

Shall I proceed?

kid51

follow-up: ↓ 7   Changed 13 years ago by coke

No reason not to proceed, sure.

in reply to: ↑ 6   Changed 13 years ago by jkeenan

Replying to coke:

No reason not to proceed, sure.

Done in r37602. Will close ticket tomorrow unless there are serious objections.

Thank you very much.
kid51

  Changed 13 years ago by jkeenan

  • status changed from assigned to closed
  • resolution set to fixed
  • patch set to applied

No objections raised. Closing ticket.

kid51

Note: See TracTickets for help on using tickets.