Labels: Type-Defect Priority-Medium
New issue 2604 by eyuw...@gmail.com: Why do we find opened files ('Affected
files') by using pattern matching instead of 'p4 opened -c CHANGESET'?
* NOTE: Do not post confidential information in this bug report. *
* If you need immediate support, please contact *
* reviewbo...@googlegroups.com *
What version are you running?
What's the URL of the page containing the problem?
No particular URL. It is a question about postreview.py in rbtools
What steps will reproduce the problem?
In postrevew.py, I see this code:
2190 if cl_is_pending and (v < 2002 or (v == "2002" and v
2191 or changenum == "default"):
2192 # Pre-2002.2 doesn't give file list in pending changelists,
2193 # or we don't have a description for a default changeset,
2194 # so we have to get it a different way.
2195 info = execute(["p4", "opened", "-c", str(changenum)],
2198 if len(info) == 1 and info.startswith("File(s) not
opened on this client."):
2199 die("Couldn't find any affected files for this
2201 for line in info:
2202 data = line.split(" ")
2203 description.append("... %s %s" % (data, data))
2206 # Get the file list
2207 for line_num, line in enumerate(description):
2208 if 'Affected files ...' in line:
2211 # Got to the end of all the description lines and
2212 # what we were looking for.
2213 die("Couldn't find any affected files for this
So any p4d process that is after 2002.2 is getting the changed file list by
searching for 'Affected files ...' in output from p4 describe -s. Why can't
you just get the files directly from p4 opened -c?
With the current way of getting file list, it is possible to error out if
someone put the string 'Affect files ...' in the change description they
put in the perforce form.
You received this message because you are subscribed to the Google Groups
To post to this group, send email to email@example.com.
To unsubscribe from this group, send email to
For more options, visit this group at