Re: [perl #46503] [TODO] Remove individual runcore command line flags...
[email protected] (Dany Arbelo) Thu, 12 Feb 2009 18:40:55 -0200
| Newsgroups | perl.perl6.internals |
|---|---|
| Message-ID | <[email protected]> |
On Wed, Feb 11, 2009 at 5:42 PM, Will Coleda via RT < [email protected]> wrote: > On Fri Oct 19 17:20:38 2007, particle wrote: > > i suggest we migrate all existing code to the new options at once, and > > mark the old options as deprecated. however, i'd like to see the old > > options stay around for a while, perhaps with a runtime warning after > > the 0.5.0 release, a fatal error after 0.6.0, and removal in 0.7.0. > > those last two release numbers are flexible > > Adding as a DEPRECATION item so we can rip these out before 1.0 The attached patch adds the mentioned deprecation warnings to this options. "make test" continues to pass. -- Daniel Arbelo Arrocha
warn_deprecated_bCfgjS_opts.patch
(application/octet-stream, 2.1 KB)
Index: compilers/imcc/main.c
===================================================================
--- compilers/imcc/main.c (revision 36509)
+++ compilers/imcc/main.c (working copy)
@@ -413,6 +413,9 @@
"\n\nhelp: parrot -h\n", opt.opt_arg);
break;
case 'b':
+ Parrot_warn(interp, PARROT_WARNINGS_ALL_FLAG,
+ "The -b option is deprecated, use the -R or "
+ "--runcore options instead.");
SET_FLAG(PARROT_BOUNDS_FLAG);
break;
case 'p':
@@ -425,18 +428,33 @@
SET_TRACE(PARROT_TRACE_OPS_FLAG);
break;
case 'j':
+ Parrot_warn(interp, PARROT_WARNINGS_ALL_FLAG,
+ "The -j option is deprecated, use the -R or "
+ "--runcore options instead.");
SET_CORE(PARROT_JIT_CORE);
break;
case 'S':
+ Parrot_warn(interp, PARROT_WARNINGS_ALL_FLAG,
+ "The -S option is deprecated, use the -R or "
+ "--runcore options instead.");
SET_CORE(PARROT_SWITCH_CORE);
break;
case 'C':
+ Parrot_warn(interp, PARROT_WARNINGS_ALL_FLAG,
+ "The -C option is deprecated, use the -R or "
+ "--runcore options instead.");
SET_CORE(PARROT_CGP_CORE);
break;
case 'f':
+ Parrot_warn(interp, PARROT_WARNINGS_ALL_FLAG,
+ "The -f option is deprecated, use the -R or "
+ "--runcore options instead.");
SET_CORE(PARROT_FAST_CORE);
break;
case 'g':
+ Parrot_warn(interp, PARROT_WARNINGS_ALL_FLAG,
+ "The -g option is deprecated, use the -R or "
+ "--runcore options instead.");
SET_CORE(PARROT_CGOTO_CORE);
break;
case 'd':