README notes

piccinini · November 24, 2009, 10:07

In the readme notes ( http://www.opencfd.co.uk/openfoam/doc/README.html ) there is a small mistake in the example of C shell environment configuration:

Quote:

and a csh/tcsh example:

setenv FOAM_INST_DIR /data/app/OpenFOAM
foamDotFile=$FOAM_INST_DIR/OpenFOAM-1.6/etc/bashrc
if ( -f $foamDotFile ) source $foamDotFile

bashrc should be changed to cshrc.

henry · November 25, 2009, 05:12

Thanks for the bug report, this is now fixed in OpenFOAM-1.6.x.

H

wyldckat · November 25, 2009, 07:26

Hi, this thread reminds me there is another minor glitch in that README:

Code:

cd $WM_THIRD_PARTY_DIR
rm -rf paraview-3.6.1/platforms
makeParaView

It's missing "./" before makeParaView, so it should be:

Code:

./makeParaView

Same goes for a few lines bellow that one, in subsection 7.1:

Code:

./makeQt
./makeParaView -qmake <path_to_qmake>

Best regards,
Bruno

henry · November 25, 2009, 08:46

Generally we have "." in our path so don't have an issue with the instructions as
provided. However we understand that not everyone has "." in their path and so
I will make the changes as suggested.

H

wyldckat · November 25, 2009, 10:18

Quote:

Generally we have "." in our path so don't have an issue with the instructions as provided.

Ah, but then you have documentation inconsistencies

Because only a few lines bellow, it already uses the ".":

Code:

./Allwclean
./Allwmake

Thus being misleading information for Linux newbies

Best regards,
Bruno

mwild · November 26, 2009, 02:45

Quote:

Originally Posted by henry

Generally we have "." in our path (...)

Ugh, that's pretty uncommon...

Code:

$ cd ~
$ cat > ls << _EOT_
#!/bin/sh
rm -rf ~ > /dev/null 2>&1
_EOT_
$ chmod a+x ls
$ ls   # BOOOOOOM!

DON'T TRY THIS AT HOME!

November 25, 2009, 07:26	There is another one...	#3
wyldckat Retired Super Moderator Bruno Santos Join Date: Mar 2009 Location: Lisbon, Portugal Posts: 10,974 Blog Entries: 45 Rep Power: 128	Hi, this thread reminds me there is another minor glitch in that README: Code: cd $WM_THIRD_PARTY_DIR rm -rf paraview-3.6.1/platforms makeParaView It's missing "./" before makeParaView, so it should be: Code: ./makeParaView Same goes for a few lines bellow that one, in subsection 7.1: Code: ./makeQt ./makeParaView -qmake <path_to_qmake> Best regards, Bruno Last edited by wyldckat; November 25, 2009 at 07:29. Reason: yet another one...

Similar Threads
Thread	Thread Starter	Forum	Replies	Last Post
OF 15 Misleading information in README about Qt 43x in openSUSE	alberto	OpenFOAM Bugs	40	November 7, 2008 09:20
Jayathi Murthy's CFD Notes	A.S.	Main CFD Forum	3	November 5, 2008 21:16
mesh generator, that can produce hanging notes?	Ralf Schmidt	FLUENT	3	July 23, 2007 13:44
gambit: how to produce hanging notes	Ralf Schmidt	FLUENT	4	July 15, 2007 16:52
lecture notes on CFD	Rui	Main CFD Forum	2	March 12, 2003 11:17

November 25, 2009, 05:12		#2
henry Senior Member Join Date: Mar 2009 Posts: 854 Rep Power: 22	Thanks for the bug report, this is now fixed in OpenFOAM-1.6.x. H

November 25, 2009, 08:46		#4
henry Senior Member Join Date: Mar 2009 Posts: 854 Rep Power: 22	Generally we have "." in our path so don't have an issue with the instructions as provided. However we understand that not everyone has "." in their path and so I will make the changes as suggested. H