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

resources: Fixed some errors and added some extra documentation #150

Merged
merged 2 commits into from
Sep 23, 2024

Conversation

Pretentiousness
Copy link
Contributor

Not sure if local-resource-example.py is a valid script in Gem5

In the last example configs/example/gem5_library/local-resource-example.py. I am not sure if it's a valid script in Gem5 codebase. Also added an example of directly using the environment variable

… a valid script in Gem5

In the last example configs/example/gem5_library/local-resource-example.py. I am not sure if it's a valid script in Gem5 codebase.
Also added an example of directly using the environment variable
@Pretentiousness Pretentiousness changed the title Resource: Fixed some errors. Resource: Fixed some errors Sep 20, 2024
@Pretentiousness
Copy link
Contributor Author

Pretentiousness commented Sep 20, 2024

@Harshil2107 @BobbyRBruce can you check if this is accurate?

@Pretentiousness Pretentiousness changed the title Resource: Fixed some errors resources: Fixed some errors Sep 21, 2024
@Pretentiousness Pretentiousness marked this pull request as ready for review September 23, 2024 20:27
@Pretentiousness
Copy link
Contributor Author

@BobbyRBruce can you check this PR.

@Pretentiousness Pretentiousness changed the title resources: Fixed some errors resources: Fixed some errors and added some extra documentation Sep 23, 2024
Copy link
Contributor

@Harshil2107 Harshil2107 left a comment

Choose a reason for hiding this comment

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

Thank you, for the contribution. This lgtm

@BobbyRBruce BobbyRBruce merged commit de135bc into gem5:stable Sep 23, 2024
@Pretentiousness Pretentiousness deleted the Resource-Fix-07 branch September 23, 2024 23:08
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