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

[Bug]: Multi Project dependency with "external" dependencies no longer build since the last release. #1417

Closed
jonaswre opened this issue Jan 27, 2025 · 4 comments
Labels
bug Something isn't working

Comments

@jonaswre
Copy link
Contributor

jonaswre commented Jan 27, 2025

AL-Go version

6.3

Describe the issue

Hi @freddydk,

there seems to be a regression in the latest AL-Go Version. If you have a multi project repo that has dependency to an app thats not in the same repo. The dependency will not be correctly installed.

Image

AppB2 fails to build.

Expected behavior

It should still build correctly.

Steps to reproduce

https://github.com/byndit-demo/App-B/actions/runs/12983061199/job/36204261921

Additional context (logs, screenshots, etc.)

No response

@jonaswre jonaswre added the bug Something isn't working label Jan 27, 2025
@freddydk
Copy link
Contributor

Could you try to merge the PR I just created.
This might be a consequence after no longer generating the thisbuild artifacts...

@freddydk
Copy link
Contributor

Looking at one of the older builds - the thisbuild artifacts was always generated for dependencies as well:

Image

I don't want to bring the thisbuild artifacts back - but I might need to write in the documentation that with v6.3 you might need to add this setting if you have external dependencies from internal dependencies

@jonaswre
Copy link
Contributor Author

Could you try to merge the PR I just created. This might be a consequence after no longer generating the thisbuild artifacts...

Where did you create the PR? https://github.com/microsoft/AL-Go/pulls?q=is%3Aopen+is%3Apr?

I don't want to bring the thisbuild artifacts back - but I might need to write in the documentation that with v6.3 you might need to add this setting if you have external dependencies from internal dependencies.

What setting?

@freddydk
Copy link
Contributor

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants