utils/getdeveloperlib.py: fix developer being reported for unrelated path
Currently, by following the instructions in the manual and querying for developers for a patch that changes path package/foobar the script reports both developers that have these entries in the DEVELOPERS file: F: package/foo/ F: package/foobar/ Starting from commit "afc112b0e4 utils/getdeveloperlib.py: fix issue with hasfile()" get-developers script uses os.path.abspath() and os.path.relpath(). The catch is that those functions return the absolute path and the relative path without the trailing slash. When the paths associated to a developer are then compared to the paths a patch touches, using the string.startswith(), any substring returns True, leading to developers for package/foo/ being wrongly reported for package/foobar/ . Fix this by re-adding the trailing slash after using relpath(). Signed-off-by: Ricardo Martincoski <ricardo.martincoski@gmail.com> Cc: Heiko Thiery <heiko.thiery@gmail.com> Cc: James Knight <james.d.knight@live.com> Cc: Thomas Petazzoni <thomas.petazzoni@bootlin.com> Signed-off-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
This commit is contained in:
parent
3942bb44e9
commit
29bb478a49
@ -253,6 +253,8 @@ def parse_developers():
|
||||
for f in dev_files:
|
||||
dev_file = os.path.relpath(f, brpath)
|
||||
dev_file = dev_file.replace(os.sep, '/') # force unix sep
|
||||
if f[-1] == '/': # relpath removes the trailing /
|
||||
dev_file = dev_file + '/'
|
||||
files.append(dev_file)
|
||||
elif line == "":
|
||||
if not name:
|
||||
|
Loading…
Reference in New Issue
Block a user