patchConcurrent Versions System - Patches: patch #5990, commitinfo %m format character -...

 
 

You are not allowed to post comments on this tracker with your current authentication level.

patch #5990: commitinfo %m format character - log message

Submitted by:  Yaron Yogev <yyogev>
Submitted on:  Mon 04 Jun 2007 08:19:57 AM UTC  
 
Category:  Feature (patch attached) Priority:  5 - Normal
Status:  None Privacy:  Public
Assigned to:  None Open/Closed:  Open
Fixed Release:  None Fixed Feature Release:  None

Mon 04 Jun 2007 08:19:57 AM UTC, original submission:

this patch adds a new %m format character to the commitinfo trigger to provide the log message in escaped format.

what for ?
in order to enable selection of different validation behavior (e.g. bypassing of validation in special cases), using the same commitinfo setup and the same filter.

use case:
- I have hundreds of CSV files stored in CVS, saving users data.
- I want to make a wholesale change of format to these files.
- I have already made the change to the files, ran my own checks to verify the format before committing, and now I want to commit it.
- the usual validation checks will take a very long time for this kind of commit, so I would like to skip them. However, I do not want to mess with the commitinfo file as other people continue their work on other files and I would like validation to be performed as usual for what they commit.
- in my log message, I will signal that validation is to be skipped using some keyword that the filter will recognize. example message:
change to new format
OPTION: SKIP_VALIDATION

as I have found, users sometimes include quotes inside their log messages. e.g.
"new splash screen for 'killer feature' demo"
however, shell_escape() had a bug which caused quotes and double quotes to be treated as the same.
to fix this, a fix in src/run.c to function shell_escape() is included. this will handle the case of quotes inside quotes.

Yaron Yogev <yyogev>

 

Attached Files

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by yyogev (Submitted the item)
  •  

    Do you think this task is very important?
    If so, you can add your encouragement to it.
    This task has 0 encouragements so far.

    Only logged-in users can vote.

     

     

     

    Follow 2 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2007-06-04 yyogev Attached File- => Added patch_cvs_commitinfo_log_message.diff, #12943
        Carbon-Copy- => Added -email is unavailable-

    Back to the top


    Powered by Savane 3.3