Skip to content

Conversation

ulrikstrid
Copy link

Started working on porting the CI flow to use Azure Pipelines.

I hit some error in the build I think https://strid.visualstudio.com/OSS/_build/results?buildId=275 it shows as green in the logs but there is no output in the build folder and the logs doesn't look like they should.

@bryphe
Copy link
Contributor

bryphe commented Dec 1, 2018

Sweet, thanks @ulrikstrid !

I hit some error in the build I think https://strid.visualstudio.com/OSS/_build/results?buildId=275 it shows as green in the logs but there is no output in the build folder and the logs doesn't look like they should.

I think with [email protected] the build artifacts don't end up in the _esy/default/build/... folder anymore - they're in the store. Unfortunately we don't know this path until build - we have to use like esy x which Bin.exe to find it.

I'm working around this for now by setting buildsInSource: "_build" so that the bits end up in the predictable _build/default/... path, but I know that's not the ideal solution - I'm sure @andreypopp has a better solution.

@ulrikstrid
Copy link
Author

I think I tried setting that as well, unfortunately the build fails locally on my windows machine. I could try building it on my Mac to figure out paths.

@bryphe
Copy link
Contributor

bryphe commented Dec 1, 2018

Yes, the path would need to be updated for the tasks we run on Windows (see my PR for an example: #199)

@ulrikstrid
Copy link
Author

Used esy x which Bin to get the path but it still can't find it for the artifact.

https://strid.visualstudio.com/OSS/_build/results?buildId=281

@kaylangan
Copy link

I'm a Program Manager on Azure Pipelines. Let me know if you have any questions or suggestions. I'm happy to help!

@ulrikstrid
Copy link
Author

@kaylangan it would be really nice to have a task that runs a script and saves the output as a variable

- script: |
     echo '##vso[task.setvariable variable=binPath]'$(esy x which Bin)
   displayName: "Save binPath variable" 

Could be more readable, but is not a blocker

jaredly added a commit that referenced this pull request Jun 25, 2019
yasssss let's get this done. Supercedes #207, which seems fitting somehow
@jaredly
Copy link
Owner

jaredly commented Jun 25, 2019

Thanks for an excellent head start! I finally got this working in #307 😅

@jaredly jaredly closed this Jun 25, 2019
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.

4 participants