Create SWI-Prolog pack (solves #4) #24
Merged
Conversation
This file contains 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
…tallation: ?- prolog_pack:valid_info_arg(version, '6.7-131003').
|
In d02da96, I had to change the version number to |
…gn packages on installation with SWI-Prolog
|
Thanks! I did some casual testing and I think this could be merged, provided that we do some immediate follow-up actions, e.g.:
@tkuhn any other comments? |
|
Sounds good. I will try to find time to look into it too, but I am not sure when that will happen. In the meantime, feel free to move ahead with this. |
Kaljurand
added a commit
that referenced
this pull request
Jul 14, 2018
Kaljurand
added a commit
that referenced
this pull request
Aug 15, 2018
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.
I wanted to use APE as a pack in SWI-Prolog. Following the instructions, I did the following:
/pack.pl. You should adapt it to your needs. Additional fields are possible, they are documented here./runape.plhave been moved into a new/prologdirectory. The original/get_ape_results.plhas been moved to/prolog/ape.pland is the entry point for SWI-Prolog'slibrary(ape)./get_ape_results.plwhich re-exports all of the predicates of/prolog/ape.pl.make all,make checkandmake install. They have been added to the Makefile.Before merging this, you should change
/pack.plto your needs. Thereafter, it should be possible to install APE by calling?- pack_install('https://github.com/Attempto/APE.git')., although I am not sure about the capitalizedAPEin the repository's name (since we are later using it with?- use_module(library(ape)).).I am not sure about
/runape.pl. I had the impression it was replaced some time ago withape.pl, but/run.shis still referencing it, while the Makefile and/make_exe.shrefer to different files.This addresses #4.