Skip to content

fix MOAB options for v5.0 #521

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.

Sign up for GitHub

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

Merged
ljacobson64 merged 4 commits into from
Dec 19, 2017
Merged

fix MOAB options for v5.0 #521

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
733f0e0
fix MOAB options for v5.0
gonuke Jul 6, 2017
bbdc3cd
suggest chmod for MCNP source in case it came from DVD
gonuke Jul 6, 2017
3378f47
Remove text referencing the --enable-dagmc option for MOAB
ljacobson64 Dec 19, 2017
542e1ab
Merge pull request #1 from ljacobson64/doc_fix
gonuke Dec 19, 2017
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 2 additions & 0 deletions doc/install/dag-mcnp.rst
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -21,13 +21,15 @@ supported versions of MCNP5 are 5.1.40, 5.1.51, 5.1.60

$ cd mcnp/mcnp5
$ cp -r <path_to_dvd>/MCNP5/Source .
$ chmod -R u+rw Source
$ patch -p0 < patch/dagmc.5.1.60.patch

Currently supported versions of MCNP6 are 6_beta2, 6.1, and 6.1.1beta.
::

$ cd mcnp/mcnp6
$ cp -r <path_to_dvd>/MCNP6/Source .
$ chmod -R u+rw Source
$ patch -p0 < patch/dagmc.6.1.1beta.patch

Assuming the patch or patches were succesfully applied, i.e. there were no
Expand Down
12 changes: 4 additions & 8 deletions doc/install/dependencies.rst
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -77,13 +77,11 @@ commands can be used to download MOAB from its `source repository
$ ln -s moab src

If you have followed the source install route, then the following commands
should be used to build MOAB. Note that the ``--enable-dagmc`` configure option
is required.
should be used to build MOAB.
::

@ljacobson64 ljacobson64 Jul 14, 2017

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Can we remove the note just above this saying that --enable-dagmc is required?


$ cd bld
$ ../src/configure --enable-dagmc \
--enable-optimize \
$ ../src/configure --enable-optimize \
--enable-shared \
--disable-debug \
--with-hdf5=$HOME/dagmc_bld/HDF5 \
Expand All @@ -93,13 +91,11 @@ is required.
$ make install

If you have followed the package manager install route, then the following
commands should be used to build MOAB. Note that the ``--enable-dagmc``
configure option is required.
commands should be used to build MOAB.
::

@ljacobson64 ljacobson64 Jul 14, 2017

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Here too


$ cd bld
$ ../src/configure --enable-dagmc \
--enable-optimize \
$ ../src/configure --enable-optimize \
--enable-shared \
--disable-debug \
--with-hdf5 \
Expand Down