Conversation
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Should I change the include path of |
No, I don't think so. Instead the variable ...
fyflags="${fyglags} -I include"
... |
auto-examples = false | ||
auto-tests = true | ||
[preprocess.cpp] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should it be
[preprocess.cpp] | |
[preprocess.fypp] |
?
As far as I remember, CPP is not used within stdlib.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, that is not used. But, we have not enabled fypp
inside fpm
as of now. So, it cannot identity it as a valid preprocessor. Considering fypp-gfortran.py
as a temporary solution, I think when we have fypp
integrated inside fpm then I think we should rename it as fypp instead of cpp.
Hi @jvdp1, I still seem to face some issues with docs ci. Can you help me in resolving this? |
This should be related to the version update of FORD, see #673 . |
@arteevraina Indeed it seems to be related to #673 |
@arteevraina Could you rebase your PR such that @zoziha chnages (#681) are also integrated into this PR, please? |
This reverts commit 16d8b86.
6269e49
to 7c99e70
Compare "PROJECT_VERSION_MAJOR=0", | ||
"PROJECT_VERSION_MINOR=1", | ||
"PROJECT_VERSION_=0", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we support extracting those values automatically in fpm?
arteevraina Sep 26, 2022 •edited
LoadingUh oh!
There was an error while loading. Please reload this page.
edited
Uh oh!
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, I think we should add support to extract them for the version number ?
I will take up this task.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PR created for this fortran-lang/fpm#766
```sh | ||
git clone https://.com/fortran-lang/fpm.git | ||
``` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why is that provided? IMO a link to fpm
should be sufficient.
This Pull Request makes few structural changes in
stdlib
to make get it compiled using the compiler wrapper script here.