Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Patch toml_error #1011

Merged
merged 1 commit into from
Mar 25, 2024
Merged

Patch toml_error #1011

merged 1 commit into from
Mar 25, 2024

Conversation

perazz
Copy link
Member

@perazz perazz commented Mar 25, 2024 •
edited
Loading

We can't have both a variable and a derive type have the same name according to:

7368775#r140199549

Here is the problematic line:

fpm/src/fpm/toml.f90

Line 259 in 7dc0171

type(toml_error), allocatable :: toml_error

This PR changes that variable name.
cc: @henilp105 @sscalpone

henilp105
henilp105 approved these changes Mar 25, 2024
View reviewed changes
Copy link
Member

@henilp105 henilp105 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @perazz , Looks good to me.

@perazz perazz merged commit 7d08e7f into main Mar 25, 2024
41 checks passed
@perazz perazz deleted the patch_toml_error branch March 25, 2024 18:07
@perazz perazz mentioned this pull request May 1, 2024
Closed
@perazz perazz mentioned this pull request Nov 30, 2024
Closed
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@henilp105 henilp105 henilp105 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@perazz @henilp105