search

open-simulation-platform / libcosimc

OSP C co-simulation API
Mozilla Public License 2.0
12 stars 2 forks source link

Use more conventional Conan channel names #17

Closed kyllingstad closed 4 years ago

kyllingstad commented 4 years ago

Accompanies open-simulation-platform/libcosim#600 to fix open-simulation-platform/libcosim#591.

It's the same change in the workflow, with an update of the channel name in conanfile.py.

kyllingstad commented 4 years ago

Looks like it's running into an issue with long path names on the Windows runners. I wonder why; if anything, this should be less of a problem now than before, since the channel name has been changed from osp/<branchname[0:50]> to osp/testing-<branchname[0:40]>, whose maximum length is actually 2 characters shorter.

hplatou commented 4 years ago

_Looks like it's running into an issue with long path names on the Windows runners. I wonder why; if anything, this should be less of a problem now than before, since the channel name has been changed from osp/<branchname[0:50]> to osp/testing-<branchname[0:40]>, whose maximum length is actually 2 characters shorter.

Might be that the 50 char limit was not enough and that it has been working since we have used shorter branch names :) We can consider using conan short_paths (https://docs.conan.io/en/latest/reference/conanfile/attributes.html#short-paths )

hplatou commented 4 years ago

_Looks like it's running into an issue with long path names on the Windows runners. I wonder why; if anything, this should be less of a problem now than before, since the channel name has been changed from osp/<branchname[0:50]> to osp/testing-<branchname[0:40]>, whose maximum length is actually 2 characters shorter.

Might be that the 50 char limit was not enough and that it has been working since we have used shorter branch names :) We can consider using conan short_paths (https://docs.conan.io/en/latest/reference/conanfile/attributes.html#short-paths )

I see you already tried it :) I think we might have a problem with the value True as the pipeline seems to parse it as a keyword and converts it to all all lower-case. Conan/Python needs it to be with capital T. Try setting either:


CONAN_USE_ALWAYS_SHORT_PATHS = 'True'
CONAN_USE_ALWAYS_SHORT_PATHS = 1
kyllingstad commented 4 years ago

Ah, I was just about to ask you if you had any idea why it didn't work. :) Thanks! :)