[Bug 1658153] Review Request: wdune - wdune (white_dune) is a
graphical VRML97/X3D editor
by bugzilla@redhat.com
https://bugzilla.redhat.com/show_bug.cgi?id=1658153
--- Comment #44 from Petr Menšík <pemensik(a)redhat.com> ---
I think I found possible improvement for default programs support.
I think runtime detection is required, compile-time detection of supported
programs is far from user-friendly.
Script, lets say named wdune-sound-editor could be set by default into
configure. It could contain something like:
#!/bin/sh
KNOWN_SOUND_EDITORS="audacity rezound wavesurfer gnusound sweep soundeditor"
FOUND=
for APP in ${KNOWN_SOUND_EDITORS}; do
FOUND="`command -v $APP`"
if test -n "$FOUND"
then
$FOUND $@
exit $?
fi
done
if test -z "$FOUND"
then
echo "ERROR: No supported editor was found!" 2>&1
exit 1
fi
It would run first program in list, similar to configure script. But would not
require reconfiguration when editors available are changed. EDITORS could be
initialized by configure variables if required.
--
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
5 years, 3 months
[Bug 1658153] Review Request: wdune - wdune (white_dune) is a
graphical VRML97/X3D editor
by bugzilla@redhat.com
https://bugzilla.redhat.com/show_bug.cgi?id=1658153
--- Comment #43 from Petr Menšík <pemensik(a)redhat.com> ---
> white_dune do not use Eigen3, it only uses vcglib without any eigen
> requirement. Maybe it is not a good idea to split the vcglib include
> files in new fedora package.
Well it does requires it as dependency. Bundled eigenlib must not be used if
there is already package for the same project. If vcglib had pkgconfig file
installed, it could have use Requires: eigen3 inside.
They you would have to only use pkg-config --cflags vcglib, and it would
include also all cflags required from eigen3 automatically.
vcglib-devel should have Requires: eigen3-devel
Because no pkgconfig file is shipped by vcglib-devel, wdune has to mention it
explicitly to compile. Fix it by shipping vcglib.pc in vcglib-devel and it can
be used in more simple way. Until that, just include eigen3 in build flags.
--
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
5 years, 3 months