Skip to content

Conversation

@awvwgk
Copy link
Member

@awvwgk awvwgk commented Aug 4, 2021

Closes #430

@awvwgk awvwgk added the specification Issue regarding fpm manifest and model label Aug 4, 2021
Copy link
Member

@milancurcic milancurcic left a comment

Choose a reason for hiding this comment

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

Looks good, thanks!

Copy link
Member

@everythingfunctional everythingfunctional left a comment

Choose a reason for hiding this comment

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

Seems reasonable to me. I'll be curious to see what people do with it.

@arjenmarkus
Copy link
Member

arjenmarkus commented Aug 5, 2021 via email

@awvwgk
Copy link
Member Author

awvwgk commented Aug 5, 2021

I was thinking about having an example, but I don't want to raise the expectation that some tool actually supports this already.

So here is a hypothetical table for the findent tool:

[extra.findent]
start-indent = 0         # <n>: starting     indent (default: 0)                      
indent = 4               # <n>: all          indents (default: 3)
indent-associate = 4     # <n>: ASSOCIATE    indent (default: all)
indent-block = 4         # <n>: BLOCK        indent (default: all)
indent-do = 4            # <n>: DO           indent (default: all)
indent-if = 4            # <n>: IF           indent (default: all)
indent-enum = 4          # <n>: ENUM         indent (default: all)
indent-forall = 4        # <n>: FORALL       indent (default: all)
indent-interface = 4     # <n>: INTERFACE    indent (default: all)
indent-module = 4        # <n>: MODULE       indent (default: all)
indent-procedure = 4     # <n>: FUNCTION, SUBROUTINE and PROGRAM indent (default: all)
indent-select = 4        # <n>: SELECT       indent (default: all)
indent-type = 4          # <n>: TYPE         indent (default: all)
indent-where = 4         # <n>: WHERE        indent (default: all)
indent-critical = 4      # <n>: CRITICAL     indent (default: all)
indent-changeteam = 4    # <n>: CHANGE TEAM  indent (default(all)
indent-continuation = 4  # <n>: continuation indent except for lines starting with '&' free to free only
indent-case = -2         # <n>: CASE         indent (default: -all/2)
indent-contains = -2     # <n>: CONTAINS     indent (default: -all/2)
indent-entry = -2        # <n>: ENTRY        indent (default: -all/2)

Or a hypothetical configuration for fpm-search adding registries for the current project

[extra.fpm.search]
label1 = "https://my-alternate-registry/index.json"
label2 = "https://other-registry/index.json"

@arjenmarkus
Copy link
Member

arjenmarkus commented Aug 6, 2021 via email

@LKedward
Copy link
Member

LKedward commented Aug 9, 2021

Thanks Sebastian @awvwgk, LGTM 👍 With two approvals and myself, I'll now merge.

@LKedward LKedward merged commit d72d953 into fortran-lang:master Aug 9, 2021
kubajj added a commit to kubajj/fpm that referenced this pull request Aug 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

specification Issue regarding fpm manifest and model

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Revisit extra table in package manifest

5 participants