Skip to content

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

Merged
mosteo merged 1 commit into
alire-project:masterfrom
AJ-Ianozi:adaflag-workaround
Aug 18, 2022
Merged

workaround for gnatcoll-core#64 / gnatstudio#128#1141
mosteo merged 1 commit into
alire-project:masterfrom
AJ-Ianozi:adaflag-workaround

Conversation

@AJ-Ianozi

Copy link
Copy Markdown
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

mosteo commented Aug 18, 2022

Copy link
Copy Markdown
Member

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
Copy Markdown
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