Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Summarize build toolchain workflow and implied rules #360

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

Merged
awvwgk merged 2 commits into fortran-lang:master from awvwgk:building-workflow
Apr 18, 2021

Conversation

@awvwgk
Copy link
Member

@awvwgk awvwgk commented Mar 22, 2021
edited
Loading

This patch will be part of a series of patches to improve the overall developer documentation.

The build tools are currently undocumented, this patch is an effort to summarize the developer workflow required to interact with the CMake and make build toolchains. It also attempts to summarize the implied rules for those toolchains by inspection of the build files, the commit history and the issue tracker. This process is not perfect due missing internal knowledge of almost a year on those files and might have resulted in a wrong perception on some of the implied rules. Feedback of previous contributors to those files is welcome.

Related to #351

LKedward reacted with thumbs up emoji
@awvwgk awvwgk added the documentation Improvements or additions to documentation label Mar 22, 2021
Copy link
Member

@jvdp1 jvdp1 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 fr these explanations. Here are some comments

Co-authored-by: Jeremie Vandenplas <jeremie.vandenplas@gmail.com>
@awvwgk awvwgk added the reviewers needed This patch requires extra eyes label Apr 17, 2021
Copy link
Member

@LKedward LKedward left a comment

Choose a reason for hiding this comment

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

This is a really useful explanation of the build system for newcomers. LGTM 👍

Copy link
Member

@jvdp1 jvdp1 left a comment

Choose a reason for hiding this comment

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

LGTM. Thank you!

@awvwgk awvwgk removed the reviewers needed This patch requires extra eyes label Apr 17, 2021
Copy link
Member Author

awvwgk commented Apr 18, 2021

Thanks for the feedback. I'll go ahead and merge.

@awvwgk awvwgk merged commit ac0f957 into fortran-lang:master Apr 18, 2021
@awvwgk awvwgk deleted the building-workflow branch April 18, 2021 12:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jvdp1 jvdp1 jvdp1 approved these changes

@LKedward LKedward LKedward approved these changes

Assignees

No one assigned

Labels

documentation Improvements or additions to documentation

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

AltStyle によって変換されたページ (->オリジナル) /