Repository navigation
Internal modules problem with $ in identifiers on AIX #2272
Description
Activity
I think it's fine. I chose
$instead of_in the first place to avoid possible collisions, it's rather unlikely that we will have$in paths, but in generally it's not a big deal, I think.- changed the title
[-]Internal modules problem with $ in identifiers[/-][+]Internal modules problem with $ in identifiers on AIX[/+]on Jul 30, 2015 Thanks, I'll include that change when I submit the PR to add AIX support
I think this can be closed for now, with discussion continuing on #2364. It can always be reopened, of course.
- added 4 commits that reference this issue
on Sep 15, 2015
Support for internal modules was added in:
2db758c
As part of this, this line in tools/utils.py converts a path name into an identifier
escaped_id = id.replace('/', '$')Unfortunately support for a $ in an identifier is implementation specific and is not supported on AIX which causes compiler failures on the generated node_natives.h file.
I'd propose to change the line to:
escaped_id = id.replace('/', '_')@vkurchatkin comments ?