Message153813
Hmm... it looks like mq_changed_files() duplicates a chunk of logic from
the existing changed_files() code. You can get rid of this redundancy by
replacing mq_changed_files() with a function that checks for applied MQ
patches; let's call it mq_patches_applied(). Then you can say:
if mq_patches_applied():
cmd += ' --rev qparent'
before the call to Popen in changed_files(), and leave the rest of the
function unmodified. The basic logic that the code will follow is this:
if <MQ patches applied>:
return files listed by 'hg status --added --modified --no-status --rev qparent'
else:
return files listed by 'hg status --added --modified --no-status'
The mq_patches_applied() function can be implemented by running
'hg qapplied' and checking whether it (a) prints at least one patch name,
(b) prints out nothing, or (c) gives an error. |
|
Date |
User |
Action |
Args |
2012-02-20 22:30:53 | nadeem.vawda | set | recipients:
+ nadeem.vawda, brett.cannon, ezio.melotti, eric.araujo, francismb |
2012-02-20 22:30:53 | nadeem.vawda | set | messageid: <1329777053.12.0.351972891764.issue14053@psf.upfronthosting.co.za> |
2012-02-20 22:30:52 | nadeem.vawda | link | issue14053 messages |
2012-02-20 22:30:52 | nadeem.vawda | create | |
|