r190735 - [analyzer] Put more uniqueness in scan-build's temporary directory names.

Jordan Rose jordan_rose at apple.com
Mon Sep 16 09:28:32 PDT 2013


On Sep 15, 2013, at 2:15 , Ed Schouten <ed at 80386.nl> wrote:

> 2013/9/14 Jordan Rose <jordan_rose at apple.com>:
>> Author: jrose
>> Date: Fri Sep 13 19:41:32 2013
>> New Revision: 190735
>> 
>> URL: http://llvm.org/viewvc/llvm-project?rev=190735&view=rev
>> Log:
>> [analyzer] Put more uniqueness in scan-build's temporary directory names.
>> 
>> This is necessary when running two scan-build processes in parallel. The
>> directory naming scheme is now:
>> 
>>  yyyy-MM-dd-HHmmss-PID-N
>>  2013-09-13-174210-123-1
>> 
>> where "PID" is the scan-build process ID, and "N" is a sequential counter
>> (not likely to be needed now that seconds are mangled in, but just in case).
>> 
>> PR17196, using a suggested fix from Greg Czajkowski!
> 
> If you were to pick T als a date-time delimiter, at least the first
> part of the string could be parsed as if it were ISO 8601:
> 
> http://en.wikipedia.org/wiki/ISO_8601#Combined_date_and_time_representations

I did think about that, but ultimately decided that it'd be easier to read in a filesystem with a character more commonly used as a delimiter. No one's expected to parse these except humans; the only set characteristic is that they are monotonically increasing within a scan-build process.*

Jordan

*...which, of course, will fail during daylight savings transitions, but I think that's acceptable.



More information about the cfe-commits mailing list