On Tue, Mar 30, 2010 at 07:34:13AM +0200, Julia Lawall wrote: > > PS: Would it be acceptable to extend the syntax of -inplace to be similar to > > sed (making backups optional)? Something like this: > > > > -inplace[=SUFFIX] > > > > edit files in place (makes backup if extension supplied) > > > > Having git, I'd like to drop them as backups are not needed but they make > > 'git > > status' a bit messy. > > Should I worry that someone out there is relying on the backup > functionality, and that that person is going to install a new version of > Coccinelle without looking at the list of changes?
Yeah, that would be user-friendly. > I could print a warning? In the worst case, someone not looking at the list of changes will also not read the warnings? > Or only have no backup if there is a .git directory in the directory provided > with the -dir option? There are too many SCM around to use heuristic IMHO. > Or make the change and hope for the best? How large is the user-base? That makes a probability of... nah, just kidding. About the renaming: If you rename to '-in-place' you would be even more sed-compatible ;) Maybe this name (and keeping the old for compatibility for a few versions) will do? Regards, Wolfram -- Pengutronix e.K. | Wolfram Sang | Industrial Linux Solutions | http://www.pengutronix.de/ |
signature.asc
Description: Digital signature
_______________________________________________ Cocci mailing list [email protected] http://lists.diku.dk/mailman/listinfo/cocci (Web access from inside DIKUs LAN only)
