On Sunday, October 31, 2010 08:42:54 am you wrote:
> Le 31/10/2010 01:44, Jirka Hladky a écrit :
> > Hi all,
> > since gather-topology.sh is nice script to help debug problems I was
> > thinking to add it to the rpm.
> > However, path to the lstopo is set to the absolute build path:
> > abs_top_builddir="/home/jhladky/tests/performance/hwloc/hwloc-1.0.2"
> > lstopo="$abs_top_builddir/utils/lstopo"
> > so it will no work after installation from rpm package (lstopo is in
> > /usr/bin/lstopo)
> > I would propose to modify the script to look first for installed lstopo
> > using "which" command.
> What about we first try the one in $abs_top_builddir/utils/lstopo and
> then revert back to $prefix/bin/lstopo ?
I'm sorry but I don't think that having $abs_top_builddir or $prefix in the rpm
is good idea. I would simply use "lstopo" or lstopo=`which lstopo` For
explanation please see bellow.
> By the way, should we rename gather-topology.sh into
> hwloc-gather-topology.sh during install ?
I would vote to install gather-topology.sh into the same directory as other
hwloc binaries (and lstopo in particular).
I have reviewed few shell scripts at /usr/bin which relies on another
programs. They call the program by the name without any path like this:
exec openoffice.org "$@"
My recommendation would be:
1) During install, rename gather-topology.sh on hwloc-gather-topology.sh
2) During install, remove variable abs_top_builddir all together and make
3)Create a minimalist man page (it just need to say in one or two sentences
what the command does)
I don't think that having abs_top_builddir in rpm is good idea. It will
resolve to some build directory on some build system.
is not working at the moment.
tar: Removing leading `/' from member names
tar: /tmp/a: Cannot stat: No such file or directory
tar: Exiting with failure status due to previous errors
mv: cannot stat `/tmp/tmp.6s80cAqWc2//tmp/a.tar.gz': No such file or directory
Hierarchy gathered in .//tmp/a.tar.gz and kept in /tmp/tmp.6s80cAqWc2//tmp/a/
./gather-topology.sh: line 54: .//tmp/a.tar.gz.output: No such file or
Expected topology output stored in .//tmp/a.tar.gz.output
I can fix this if you are interested.
Please let me know your opinion. Please let me know if I can help with any
work to implement the proposed changes.