Ticket #310 (closed todo: fixed)

Opened 6 years ago

Last modified 5 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 6 years ago by jkeenan

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

in reply to: ↑ description   Changed 6 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 5 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 5 years ago by jkeenan

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

in reply to: ↑ 3   Changed 5 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 5 years ago by coke

No reason not to proceed, sure.

in reply to: ↑ 6   Changed 5 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 5 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.