jfrog-azure-devops-extension icon indicating copy to clipboard operation
jfrog-azure-devops-extension copied to clipboard

conan install pathOrReference is not parsed correctly

Open DdOtzen opened this issue 5 years ago • 0 comments

Describe the bug Command fails if reference contains user/channel

To Reproduce set

 pathOrReference:  . aUser/aChannel

Expected behavior install completes with package setup as given user/channel

Versions Task : Artifactory Conan Description : This task runs a Conan command. Version : 1.11.2 Author : JFrog

Running in docker container build on ubuntu 18 TS

Additional context Looking in the source it seems that the problem is that the 'pathOrReference' is simply pushed to the array 'conanArguments' at /tasks/ArtifactoryConan/conanBuild.js#L149 unlike the 'extraArguments' that is split into array members by addExtraArguments()

I have confirmed this somewhat by using this workaround instead of failing setup

 pathOrReference:  . aUser/aChannel

I use

      extraArguments: .
      pathOrReference:  aUser/aChannel

This is not obvious as these output lines:

Running Conan command at: /__w/1/p400_dsp
/usr/local/bin/conan install . aUser/aChannel

is exactly the same for boyh failing setup and the workaround setup.

Only by looking in the code I have to conclude that what conan sees on the commandline is similar to for fail setup:

/usr/local/bin/conan install ". aUser/aChannel"

for workaround setup

/usr/local/bin/conan install "." "aUser/aChannel"

DdOtzen avatar Jan 25 '21 16:01 DdOtzen