Skip to content

#270 Look for MAPPING.CONFIG in the same directory as genattrib.py using absolute paths - #271

Closed
JonoYang wants to merge 95 commits into
masterfrom
270-genattrib-mapping-fix
Closed

#270 Look for MAPPING.CONFIG in the same directory as genattrib.py using absolute paths#271
JonoYang wants to merge 95 commits into
masterfrom
270-genattrib-mapping-fix

Conversation

@JonoYang

Copy link
Copy Markdown
Member

The absolute path to MAPPING.CONFIG is now used by genattrib.py when the --mapping option is used.

pombredanne and others added 30 commits May 5, 2015 18:24
Add try/catch to handle when ABOUT files don't have dje_license_name.

Add code to handle long path specifically in windows (read and write)
See See https://msdn.microsoft.com/en-us/library/aa365247.aspx

Add code to strip the UNC_Prefix for path location the in the error log.

Create test for long path handling.

Add code to strip the quote from the api input.
for long path in windows.

In addition, adding code to make sure both the input and filter are in
posix path format before doing any string partition.
Add code to check if the about_file path is valid or invalid.
If the path is invalid, the tool will log the error and skip this
generation and keep continue with the rest.
* Handle input's encoding issues
* Better error handling
* Writing to and reading from Windows OS with paths > 255 chars
chinyeungli and others added 22 commits January 13, 2016 12:48
The tool will not log an error in the error.log if a directory name in a
file path endswith space.
Missing version value will not treat as a warning instead of error.
Linux Installation - Directory name (Minor)
#267 MAPPING.CONFIG not found when calling genabout.py from different directory
… prepending the absolute path to the directory genattrib.py is to 'MAPPING.CONFIG'
@pombredanne

Copy link
Copy Markdown
Member

@JonoYang @chinyeungli why has this been merged in?

@pombredanne

Copy link
Copy Markdown
Member

@JonoYang is this still relevant or stale and should be closed?

@pombredanne

Copy link
Copy Markdown
Member

Note that if you still want to merge this will have to go in the v2-maintenance branch as this was branched off the v2 master

@pombredanne

Copy link
Copy Markdown
Member

This is stale and only for v2-maintainance.... c87c1ff is the only relevant commit anyway. I am closing this branch

@pombredanne
pombredanne deleted the 270-genattrib-mapping-fix branch November 16, 2018 08:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants