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

configure: fix for issue #112 #113

Merged
merged 1 commit into from
Nov 8, 2023

Conversation

gcongiu
Copy link
Contributor

@gcongiu gcongiu commented Nov 8, 2023

Pull Request Description

Addresses issue #112

Author Checklist

  • Description
    Why this PR exists. Reference all relevant information, including background, issues, test failures, etc
  • Commits
    Commits are self contained and only do one thing
    Commits have a header of the form: module: short description
    Commits have a body (whenever relevant) containing a detailed description of the addressed problem and its solution
  • Tests
    The PR needs to pass all the tests

@wcohen
Copy link
Contributor

wcohen commented Nov 8, 2023

Yes, this patch does fix the issue with the configure. The rpmbuild of the papi tarball works without any patching of configure on this branch.

@gcongiu
Copy link
Contributor Author

gcongiu commented Nov 8, 2023 via email

@gcongiu gcongiu added this to the PAPI 7.1.0 release milestone Nov 8, 2023
@gcongiu gcongiu requested a review from adanalis November 8, 2023 13:04
@gcongiu gcongiu force-pushed the 2023.11.08_issue-112-fix branch from bb0df81 to 1c1d00d Compare November 8, 2023 13:05
@gcongiu gcongiu force-pushed the 2023.11.08_issue-112-fix branch from 1c1d00d to 6ce9f07 Compare November 8, 2023 14:53
@gcongiu gcongiu merged commit a182687 into icl-utk-edu:master Nov 8, 2023
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