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

Makefile: remove left over compile instructions for flux_account_shares #482

Merged
merged 1 commit into from
Aug 8, 2024

Conversation

cmoussa1
Copy link
Member

@cmoussa1 cmoussa1 commented Aug 7, 2024

Problem

flux_account_shares.cpp was removed in #471, but there is still a compile definition for it in the top-level Makefile.am that is left over.


This PR just removes the left over compilation instructions.

Fixes #481

Problem: flux_account_shares.cpp was removed in flux-framework#471, but there is
still a compile definition for it in the top-level Makefile.am that
is left over.

Remove the left over compilation instructions.
@cmoussa1 cmoussa1 requested a review from jameshcorbett August 7, 2024 21:34
Copy link
Member

@jameshcorbett jameshcorbett left a comment

Choose a reason for hiding this comment

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

💯

@cmoussa1
Copy link
Member Author

cmoussa1 commented Aug 8, 2024

🙏 setting MWP here

@mergify mergify bot merged commit aa4019f into flux-framework:master Aug 8, 2024
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

src/Makefile.am: remove flux_account_shares targets
2 participants