Implement fpm publish
#876
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…s, get system tmp directory, read command line outputs
# Conflicts: # src/fpm/git.f90
Closed
# Conflicts: # src/fpm_command_line.f90
arteevraina
approved these changes
Apr 20, 2023
minhqdao
commented
Apr 21, 2023
| @@ -1020,4 +1021,65 @@ subroutine get_home(home, error) | |||
| end if | |||
| end subroutine get_home | |||
|
|
|||
| !> Execute command line and return output as a string. | |||
There was a problem hiding this comment.
I think this should be built into fortran just as execute_command_line. I was searching whether we already have a routine implemented in fpm but couldn't find any. Have I missed sth? @awvwgk @everythingfunctional @milancurcic
There was a problem hiding this comment.
I haven't found any better method of capturing external command output in Fortran. I could potentially mention it to the standards committee.
There was a problem hiding this comment.
I think it would be great if we had that.
minhqdao
commented
Apr 21, 2023
Comment on lines
+1060
to
+1083
| !> Get system-dependent tmp directory. | ||
| subroutine get_tmp_directory(tmp_dir, error) | ||
| !> System-dependant tmp directory. | ||
| character(len=:), allocatable, intent(out) :: tmp_dir | ||
| !> Error to handle. | ||
| type(error_t), allocatable, intent(out) :: error | ||
|
|
||
| tmp_dir = get_env('TMPDIR', '') | ||
| if (tmp_dir /= '') then | ||
| tmp_dir = tmp_dir//'fpm'; return | ||
| end if | ||
|
|
||
| tmp_dir = get_env('TMP', '') | ||
| if (tmp_dir /= '') then | ||
| tmp_dir = tmp_dir//'fpm'; return | ||
| end if | ||
|
|
||
| tmp_dir = get_env('TEMP', '') | ||
| if (tmp_dir /= '') then | ||
| tmp_dir = tmp_dir//'fpm'; return | ||
| end if | ||
|
|
||
| call fatal_error(error, "Couldn't determine system temporary directory.") | ||
| end |
There was a problem hiding this comment.
Same for this actually. @awvwgk @everythingfunctional @milancurcic
There was a problem hiding this comment.
This one is built in.
error stop "Couldn't ..."
5 tasks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Implement api for publishing a package to the registry.
fpm_publish_settingsandfpm_cmd_publishmodule.--show-form-datainstead of publishing.--show-versioninstead of publishing.gitdependencies.git archive.git archive -land use available method.curl.How to test
Run
fpm publish --show-package-versionto print the package version to the console.Run
fpm publish --show-form-datato print the form data to the console that would otherwise be sent throughfpm publish --token <token>. If you include the token (fpm publish --token <token> --show-form-data), the token will be included in the form data.Get a (test) token from the registry website and run
fpm publish --token <token>to publish a package to the (test) registry.