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

Problem using gnoga with Alire #10

Closed
bracke opened this issue May 28, 2024 · 7 comments
Closed

Problem using gnoga with Alire #10

bracke opened this issue May 28, 2024 · 7 comments

Comments

@bracke
Copy link
Contributor

bracke commented May 28, 2024

There is a problem when trying to add gnoga to a alr proejct.

alr init --bin mytestapp
cd mytestapp
alr build
--> builds successfull

alr with gnoga
alr build
-->
error: Trying to set an already defined environment variable:
error: Target_OS is already defined as 'Linux' but new value is 'auto'

I have reported this to thew Alire project alire-project/alire#1687 but apparently it is a problem with Gnoga:

Thanks for the report. This is indeed a mistake in the metadata of both gnoga and simple_components, that should not set these variables.

@Blady-Com
Copy link
Owner

Alire support for Gnoga is actually in an exploring access mode as building app processes are different.
With Gnoga you create your app inside Gnoga file structure.
With Alire you create your app in its own file structure using components.
That being said, I'm interested to know how you build you app in order to make relevant changes.
Could you share your project?
In the mean time, I suggest to follow the proposal of mosteo, at your project root do:
% alr pin simple_components=4.62.0
and then
% alr build
should be ok.

@bracke
Copy link
Contributor Author

bracke commented May 30, 2024

@Blady-Com

Could you share your project?:

I*m just experimenting, so I literally just did what I mentioned in the issue description:

alr init --bin mytestapp
cd mytestapp
alr build
--> builds successfull

alr with gnoga
alr build
-->
error: Trying to set an already defined environment variable:
error: Target_OS is already defined as 'Linux' but new value is 'auto'

I will try with the pin.

@mosteo
Copy link

mosteo commented May 30, 2024

I will fix this one in the Alire index directly.

@mosteo
Copy link

mosteo commented May 30, 2024

Fixed in alire-project/alire-index#1082

@bracke
Copy link
Contributor Author

bracke commented May 30, 2024

@mostea
Should it work now?
I have tried alr index --update-all and then repeated the above but the result is the same.

@mosteo
Copy link

mosteo commented May 31, 2024

Sorry, it was not yet merged. Please try it now.

@bracke
Copy link
Contributor Author

bracke commented May 31, 2024

@mosteo I can confirm it works now.
Thank you for the quick response.

@bracke bracke closed this as completed May 31, 2024
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

3 participants