View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
0002376 | Slicer4 | Core: Base Code | public | 2012-07-28 20:15 | 2017-06-07 23:27 |
Reporter | fedorov | Assigned To | jcfr | ||
Priority | normal | Severity | minor | Reproducibility | always |
Status | closed | Resolution | fixed | ||
Product Version | |||||
Target Version | Slicer 4.3.0 | Fixed in Version | Slicer 4.3.0 | ||
Summary | 0002376: Inconsistent behavior of Slicer launcher command options output across systems | ||||
Description | On Windows "$Slicer --settings-path" does not produce output. Also, "Slicer --help" output is different from On Mac: [fedorov@bear Slicer-build] On Windows in cygwin: $ ./Slicer.exe --help Options Options On Windows in "cmd" "Slicer --help" does not produce any output. I am testing with 2012-07-25 nightly build on Windows 7. On this | ||||
Tags | No tags attached. | ||||
related to | 0001683 | closed | jcfr | ctk app launcher doesn't handle windows command console |
related to | 0002917 | closed | jcfr | Ensure --cmd option is available in installed package |
parent of | 0002934 | assigned | jcfr | Console output logging for Windows |
related to | 0002691 | closed | jcfr | No output messages in the slicer command |
Not all the children of this issue are yet resolved or closed. |
For the reference, there was no popup dialog with the message. |
|
--settings-path issue fixed in r21210 Output will now be consistent between built tree and installed tree. Fixed in r21211 Duplicated output on the launcher help will be fixed with the coming release of the launcher. See https://github.com/commontk/AppLauncher/issues/32 |
|
Another inconsistency: on Windows, Slicer detaches itself from cmd terminal on startup. Also, there is no error log, and if Slicer crashes, there is no way to know what happened. Can this be resolved at some point? |
|
When using the installed binaries, by default it's expected to have the main executable detach from the console. You should be able to keep attach by using the "--no-splash" argument. |
|
Checked with Oct 5 binary (what I have installed on my VirtualBox Win7), slicer is detached from the console when I use --no-splash argument. |
|
Good point. On windows installed tree, the version of the launcher build without IO support is used. The entry point "WinMain" is called [1] and is defacto detached from the console. Providing a SlicerWithOutput executable would allow to obtain the output. [1] https://github.com/commontk/AppLauncher/blob/master/Main.cpp#L123 |
|
Potentially related issue? http://massmail.spl.harvard.edu/public-archives/slicer-devel/2012/010375.html Hi, I have a script that I am running from command line using If I run it without --no-splash, I have Slicer crash. With Perhaps it could be relevant that the script produces a lot of console AF |
|
Consistency of launcher has been addressed with version 0.1.9 Output on windows will be addressed while working on issue 0001683 Consider also reading the following note referencing a topic were preliminary work has been done: http://www.na-mic.org/Bug/view.php?id=1683#c6686 |
|
From http://massmail.spl.harvard.edu/public-archives/slicer-devel/2013/011259.html ==> Francois Budin: Thanks Andriy, I don't think that it has been fixed. It seems to be targeted for Francois On 02/06/2013 12:39 PM, Andriy Fedorov wrote:
|
|
As a side note, there is a Slicer_BUILD_WIN32_CONSOLE cmake option that can be set to ON at configure time. |
|
On windows, by design the distributed application is built with no console output. See [1], [2], [3] and [4]. For windows, 2 flavor of the launcher are available [4]:
Since the inconsistency reported in issue 2376 have been fixed and there are no console output by design, I am thinking to resolve the issue. We should probably open a new issue described as: "On windows, propose a mechanism allowing to report textual information associated with command line switch when Slicer is build within console IO support" In other word, when the following command are issued: For this set of option requiring console output, what should happen ?
[1] https://github.com/Slicer/DashboardScripts/blob/master/factory-south-win7-vs2008-64bits_slicer_422_release_package.cmake#L49 |
|
From Francois: I think even just a log file would be very helpful to know if and what went wrong. Otherwise, being able to just open a new terminal with the environment set up would be great too. |
|
From Andriy: I agree with Francois. I am perfectly fine with inconsistent behavior |
|
Inconsistency has been already addressed. Issue 0002917 capture the fact "--cmd" option is not available from an installed Slicer. |
|
Fix committed to 1683-launcher-with-output branch. |
|
Slicer: 1683-launcher-with-output 5b08d1b4 2012-10-18 15:09:59 Details Diff |
ENH: Using "--settings-path" output text an all platform See http://www.na-mic.org/Bug/view.php?id=2376 Fixes 0002376 git-svn-id: http://svn.slicer.org/Slicer4/trunk@21210 3bd1e089-480b-0410-8dfb-8563597acbee |
||
mod - Applications/SlicerApp/CMakeLists.txt | Diff File | ||
Slicer: 1683-launcher-with-output ad584f51 2012-10-18 15:10:01 Details Diff |
BUG: Display "Usage" and "Options" when using installed MacOSX Slicer executable See http://www.na-mic.org/Bug/view.php?id=2376 Fixes 0002376 git-svn-id: http://svn.slicer.org/Slicer4/trunk@21211 3bd1e089-480b-0410-8dfb-8563597acbee |
||
mod - Base/QTCore/qSlicerCoreApplication.cxx | Diff File | ||
Slicer: 1683-launcher-with-output 16d828ff 2012-10-22 17:41:38 Details Diff |
BUG: Fix CTK AppLauncher to have consistent output across platform. * Now uses CTK AppLauncher 0.1.9 * Note that doesn't fix issue 0001683. See http://www.na-mic.org/Bug/view.php?id=1683 See http://www.na-mic.org/Bug/view.php?id=2376 Fixes 0002376 git-svn-id: http://svn.slicer.org/Slicer4/trunk@21223 3bd1e089-480b-0410-8dfb-8563597acbee |
||
mod - SuperBuild/External_CTKAPPLAUNCHER.cmake | Diff File |
Date Modified | Username | Field | Change |
---|---|---|---|
2012-07-28 20:15 | fedorov | New Issue | |
2012-07-28 20:15 | fedorov | Status | new => assigned |
2012-07-28 20:15 | fedorov | Assigned To | => pieper |
2012-07-29 06:38 | pieper | Assigned To | pieper => jcfr |
2012-07-31 12:39 | fedorov | Relationship added | related to 0001683 |
2012-07-31 12:44 | fedorov | Note Added: 0005471 | |
2012-08-21 12:58 | jcfr | Target Version | => Slicer 4.3.0 |
2012-10-18 11:13 | jcfr | Note Added: 0006621 | |
2012-10-18 17:38 | fedorov | Note Added: 0006634 | |
2012-10-19 07:23 | jcfr | Note Added: 0006641 | |
2012-10-19 07:25 | fedorov | Note Added: 0006642 | |
2012-10-22 10:28 | jcfr | Note Added: 0006681 | |
2012-10-24 11:20 | fedorov | Note Added: 0006726 | |
2012-10-24 11:25 | jcfr | Note Added: 0006728 | |
2012-10-24 11:25 | jcfr | Status | assigned => resolved |
2012-10-24 11:25 | jcfr | Fixed in Version | => Slicer 4.2.0 - coming release |
2012-10-24 11:25 | jcfr | Resolution | open => fixed |
2012-10-29 08:25 | fedorov | Relationship added | related to 0002691 |
2013-02-06 08:25 | fedorov | Note Added: 0007827 | |
2013-02-06 08:25 | fedorov | Status | resolved => feedback |
2013-02-06 08:25 | fedorov | Resolution | fixed => reopened |
2013-02-06 08:30 | finetjul | Note Added: 0007828 | |
2013-02-06 09:01 | jcfr | Note Added: 0007829 | |
2013-02-06 09:26 | jcfr | Note Added: 0007830 | |
2013-02-06 09:37 | jcfr | Relationship added | related to 0002917 |
2013-02-06 09:41 | jcfr | Note Added: 0007831 | |
2013-02-06 09:42 | jcfr | Note Added: 0007832 | |
2013-02-06 09:42 | jcfr | Status | feedback => resolved |
2013-02-06 09:42 | jcfr | Resolution | reopened => fixed |
2013-02-10 03:42 | fedorov | Status | resolved => closed |
2013-02-11 09:15 | fedorov | Relationship added | parent of 0002934 |
2013-02-12 09:40 | jcfr | Fixed in Version | Slicer 4.2.0 => Slicer 4.3.0 |
2017-06-07 23:27 | jcfr | Changeset attached | => Slicer 1683-launcher-with-output 16d828ff |
2017-06-07 23:27 | jcfr | Changeset attached | => Slicer 1683-launcher-with-output ad584f51 |
2017-06-07 23:27 | jcfr | Changeset attached | => Slicer 1683-launcher-with-output 5b08d1b4 |
2017-06-07 23:27 | jcfr | Note Added: 0014539 |