Guy Harris skrev 2011-08-29 21:15:
On Aug 29, 2011, at 10:48 AM, Stephen Fisher wrote:
On Mon, Aug 29, 2011 at 10:05:33AM +0200, Anders Broman wrote:
On Linux trying to load the UI description from file the file will be
loaded from someting like /usr/local/share/wireshark/ui/ runing
Wireshark from the working directory will not find the file unless
Wireshark is installed (or the file manually copied). Any suggestions
on how to resolve/improve this?
Some things, such as tools/fuzz-test.sh define the environment variable
WIRESHARK_RUN_FROM_BUILD_DIRECTORY=1. Can this be tested for to tell
Wireshark where to load the files from?
What should be tested is the running_in_build_directory_flag Boolean. In many cases, that's already tested. However, if the source directory isn't laid out the way the install directory is - which is the case for the UI description files, which are in the source directory gtk/ui but are installed in the directory ui - the code that figures out the path needs to call running_in_build_directory() to figure out whether to search in gtk/ui or just ui, relative to the result of get_datafile_dir().
I.e.:
gui_desc_file_name = g_strdup_printf("%s" G_DIR_SEPARATOR_S %s G_DIR_SEPARATOR_S "tree-view-ui.xml", get_datafile_dir(),
running_in_build_directory() ? "gtk/ui" : "ui");
gtk_ui_manager_add_ui_from_file(ui_manager_tree_view_menu, gui_desc_file_name,&error);
or, even better:
gchar *
get_ui_file_path(const char *filename)
{
gchar *gui_desc_file_name;
gui_desc_file_name = g_strdup_printf("%s" G_DIR_SEPARATOR_S %s G_DIR_SEPARATOR_S "%s", get_datafile_dir(),
running_in_build_directory() ? "gtk/ui" : "ui", filename);
return gui_desc_file_name;
}
and use that in all the places where stuff is loaded from a UI description file.
running_in_build_directory_flag is set to TRUE if the WIRESHARK_RUN_FROM_BUILD_DIRECTORY environment variable is set.
On UN*X, it's also set if argv[0] contains the string "/.libs", so that it's automatically set if what you're running on the command line is the wrapper script built by libtool, running a binary in the .libs directory; this means you should rarely need to set WIRESHARK_RUN_FROM_BUILD_DIRECTORY yourself.
On Windows, that doesn't work. For most paths, it doesn't matter, because the Windows binary assumes the data files are installed in the same directory tree as the executable image, but for the cases where the source directory isn't laid out the way the install directory is, it does a hack^Wheuristic, wherein if the purported directory either doesn't exist or isn't a directory, it assumes you're running from the build directory. For example, the init_wspython_dir() and init_plugin_dir() routines do that; perhaps there should be an init_ui_dir() routine that behaves similarly, and a get_ui_dir() routine to return the constructed UI directory, in which case, instead of calling get_ui_file_path(), you'd do
gui_desc_file_name = g_strdup_printf("%s" G_DIR_SEPARATOR_S {file name}", get_ui_dir());
Not sure if it's an issue on windows. Makefile.nmake copies all the
executables and data files
including plugins to ../wireshark-gtk2/.. and if you run from there
everyting is OK I think.
/Anders
___________________________________________________________________________
Sent via: Wireshark-dev mailing list<wireshark-dev@xxxxxxxxxxxxx>
Archives: http://www.wireshark.org/lists/wireshark-dev
Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev
mailto:wireshark-dev-request@xxxxxxxxxxxxx?subject=unsubscribe
