Project

General

Profile

Bug #215

omniORB default configuration file

Added by Guilhem Saurel 25 days ago. Updated 24 days ago.

Status:
Closed
Priority:
Normal
Assignee:
-

Description

Hi,

In middleware/omniORB/Makefile:57, the sample.cfg file is installed in ${PREFIX}/share/examples/omniORB/. Is there a reason not to install it directly in $OMNIORB_ETCDIR ?

Ubuntu's version provides a functional configuration file by default without requiring any user's action, so this is misleading when they switch to robotpkg's deb:
https://github.com/humanoid-path-planner/hpp-corbaserver/issues/90

Best,
Guilhem.

Associated revisions

Revision 30c6b31d (diff)
Added by Anthony Mallet 24 days ago

[middleware/omniORB] Install a default configuration file

omniORB provides a sample configuration file that is installed in
share/example/omniORB. Also install it in etc/ for user convenience, where
omniORB looks by default.

In order to not overwrite or delete a user-modified file, do this via a
POST-INSTALL / DEINSTALL script that checks that the file is identical to the
sample one. If not, it is left in place during removal, and not overwritten
during install.

Also add patch-ag, adding a useful option to update default configuration
file.

Should fix #215
Bump PKGREVISION

History

#1

Updated by Anthony Mallet 25 days ago

Is the default sample.cfg OK?

Ubuntu does

@@ -394,7 +394,7 @@
 #   A call to resolve_initial_references("MyService") will return the object
 #   reference denoted by `corbaloc::myhost.example.com/MyService'.
 #
-#   The default is empty.
+DefaultInitRef = corbaloc::

Is it needed?

#2

Updated by Guilhem Saurel 25 days ago

I just checked, and it is. This might explain why I had issues with Arch… Thanks for noticing !

#3

Updated by Anthony Mallet 24 days ago

One issue with automatically putting files in etc/ is that `apt
install` seems to happily overwrite any custom user file here, and
`apt purge` will just delete the file even if it was modified.

This is not a desired behaviour for user configuration files ...

#4

Updated by Anthony Mallet 24 days ago

  • Status changed from New to Closed

Also available in: Atom PDF