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
Hydjet integration as external #5412
Merged
Merged
Changes from 9 commits
Commits
Show all changes
14 commits
Select commit
Hold shift + click to select a range
97b8793
Create hydjet.spec
wouf 4698a3e
Create hydjet-toolfile.spec
wouf 7b2cd75
Update pyquen.spec
wouf 250cebb
Merge pull request #3 from wouf/patch-5
wouf c5a125a
Merge pull request #4 from wouf/patch-4
wouf 7134a15
Update cmssw-tool-conf.spec
wouf 399308a
Merge pull request #5 from wouf/patch-7
wouf 30ddf90
Update pyquen.spec
wouf 9556f1e
Update hydjet.spec
wouf 44ed6cd
Update pyquen.spec
wouf 2c0d3be
Update hydjet.spec
wouf 75029eb
Add /doc.txt file for hydjet
mrodozov 6c673a2
Update doc.txt
wouf 301be52
Create doc.txt
wouf File filter
Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
There are no files selected for viewing
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
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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,23 @@ | ||
### RPM external hydjet-toolfile 1.0 | ||
Requires: hydjet | ||
%prep | ||
|
||
%build | ||
|
||
%install | ||
|
||
mkdir -p %i/etc/scram.d | ||
cat << \EOF_TOOLFILE >%i/etc/scram.d/hydjet.xml | ||
<tool name="hydjet" version="@TOOL_VERSION@"> | ||
<lib name="hydjet"/> | ||
<client> | ||
<environment name="HYDJET_BASE" default="@TOOL_ROOT@"/> | ||
<environment name="LIBDIR" default="$HYDJET_BASE/lib"/> | ||
</client> | ||
<use name="pyquen"/> | ||
<use name="pythia6"/> | ||
<use name="lhapdf"/> | ||
</tool> | ||
EOF_TOOLFILE | ||
|
||
## IMPORT scram-tools-post |
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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,20 @@ | ||
### RPM external hydjet 1.9.1 | ||
|
||
Source: http://cern.ch/lokhtin/hydro/%{n}-%{realversion}.tar.gz | ||
|
||
BuildRequires: cmake | ||
|
||
Requires: pyquen pythia6 lhapdf | ||
|
||
|
||
%prep | ||
%setup -q -n %{n}-%{realversion} | ||
|
||
%build | ||
|
||
cmake . -DCMAKE_INSTALL_PREFIX=%i -DCMAKE_BUILD_TYPE=Release -DPYQUEN_DIR=${PYQUEN_ROOT} -DPYTHIA6_DIR=${PYTHIA6_ROOT} -DLHAPDF_ROOT_DIR=${LHAPDF_ROOT} | ||
cmake --build . --clean-first -- %{makeprocesses} | ||
|
||
%install | ||
|
||
cmake --build . --target install -- %{makeprocesses} |
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
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.
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.
please also add build dependency on
gmake
otherwise it will pick system gmake.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.
Thanks for advice. But, does it matter?
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 it does, some times we end up in env (e.g. docker containers with minimal package installation) where gmake is not available via system
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.
I mean does it matter gmake, make, or ninja tool is used?
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.
it depends, if you have run cmake to use ninja then you need to add dependency on ninja otheriwse default is
make
(which is available via our gmake.spec) and for that you need to add dependency on gmake.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.
Thank You for Your explanation!