Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Windows absolute path #36

Open
kimstik opened this issue Aug 17, 2020 · 2 comments
Open

Windows absolute path #36

kimstik opened this issue Aug 17, 2020 · 2 comments

Comments

@kimstik
Copy link

kimstik commented Aug 17, 2020

Using ":" as separator create some issue for windows absolute path utilization .

test case:
--include unisim:%XILINX_VIVADO%/data/vhdl/src/unisims
expands into:
C:\Xilinx\Vivado\2019.1\data\vhdl\src\unisims
where C wrongly threated as unisim library version.

So version should be specified explicitly in this case as workaround.

@jvanstraten
Copy link
Collaborator

I've never tried vhdeps on Windows, so honestly I'm surprised it works at all. I can see why the colons would break on Windows, but if I understand correctly you can already solve it by doing something of the form --include 93:unisim:%XILINX_VIVADO%/data/vhdl/src/unisims? Or am I missing something?

To be more Windows-compatible a different separator than : should be used, I guess, but it's quite difficult to find one that isn't already in use for other things.

@kimstik
Copy link
Author

kimstik commented Aug 17, 2020

You got it exactly.
Colons may be ignored inside "" :
--include unisim:"%XILINX_VIVADO%/data/vhdl/src/unisims"

Other way is to fallback on "ValueError: invalid literal for int() with base 10: 'unisim'"
to "windows" mode in case of path matched to \D:/.*

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

No branches or pull requests

2 participants