Skip to content

workaround for gnatcoll-core#64 / gnatstudio#128 #1141

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

Merged
merged 1 commit into from
Aug 18, 2022
Merged

workaround for gnatcoll-core#64 / gnatstudio#128 #1141

merged 1 commit into from
Aug 18, 2022

Conversation

AJ-Ianozi
Copy link
Contributor

This is a workaround for issue AdaCore/gnatstudio#128 and AdaCore/gnatcoll-core#64 where GNAT Studio fails to open recently-updated or newly-created alire files.

The issue was fixed in AdaCore/gnatcoll-core@0553172 so if alire will be using that version of gnatcoll in their upcoming release, feel free to reject this PR. Otherwise, this does seem to fix the problem in a less drastic way than was suggested in gnatstudio#128 by breaking out the Ada_Compiler_Switches assignment into two statements. So config/<crate>_config.gpr from e.g.

   Ada_Compiler_Switches := External_As_List ("ADAFLAGS", " ") &
          (
           . . .
          );

To e.g.

   Ada_Compiler_Switches := External_As_List ("ADAFLAGS", " ");
   Ada_Compiler_Switches := Ada_Compiler_Switches &
          (
            . . .
          );

P.S. I am new to github, and this is my very first actual PR, so I apologize in advance if I did it incorrectly!

This is a workaround for issue AdaCore/gnatstudio#128 and AdaCore/gnatcoll-core#64 where GNAT Studio fails to open recently-updated or newly-created alire files. Breaking the Ada_Compiler_Switches up into two statements resloves this problem.
@mosteo
Copy link
Member

mosteo commented Aug 18, 2022

As we use our own fork of gnatcoll for the moment this will come in handy. Thanks, @AJ-Ianozi.

@mosteo mosteo merged commit 2830b1e into alire-project:master Aug 18, 2022
mosteo pushed a commit that referenced this pull request Aug 18, 2022
This is a workaround for issue AdaCore/gnatstudio#128 and AdaCore/gnatcoll-core#64 where GNAT Studio fails to open recently-updated or newly-created alire files. Breaking the Ada_Compiler_Switches up into two statements resloves this problem.
@Fabien-Chouteau
Copy link
Member

For the record, Alire is not reading gpr files at all. So Alire itself is not impacted by this.

@AJ-Ianozi AJ-Ianozi deleted the adaflag-workaround branch August 24, 2022 00:17
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.

3 participants