Change usages of 'Getopt::Obj' to ['Getopt';'Obj']

[coke]

This should be reverted until post 2.0; while the new NS is the desired one, we've removed the original.

The best solution is probably to put it into /both/ namespace for 2.0, add the deprecation notice in 2.0, and then remove the old one post 2.0.

[dukeleto]

Is this about JSON vs data_json ? The TT is ambiguous

[coke]

This is for r43083, which changed 'Getopt::Obj' to ['Getopt';'Obj']

[coke]

To be clear, this change completely breaks partcl (and any other language that is using this core library).

[coke]

I believe a fix was committed for this that makes both variants available now; Let's add in a notice to DEPRECATED.pod about removing the old one as of version 2.1.

All the usages of the old version in core should be updated to the new version and tested.

[jkeenan]

For reference, as of r43188 the only files in the repository that match for Getopt::Obj are:

./runtime/parrot/library/Getopt/Obj.pir
./t/library/getopt_obj.t

[jkeenan]

Adapted from tene's original patch in r43083

[jkeenan]

Replying to coke:

I believe a fix was committed for this that makes both variants available now; Let's add in a notice to DEPRECATED.pod about removing the old one as of version 2.1.

Coke,

Having passed the deprecation point, is the attached patch now applicable? (It passes make test; see  this Smolder report.)

Will it break partcl? Or, is avoiding breakage now the HLL's responsibility?

kid51

[coke]

Yes, you can rip out the old version now that 2.0 is out. Thanks.

[jkeenan]

Replying to coke:

Yes, you can rip out the old version now that 2.0 is out. Thanks.

Applied in r43581. Will keep ticket open for 1-3 days for any complaints.

Thank you very much.
kid51

[jkeenan]

No complaints recorded. Closing ticket.