-
Notifications
You must be signed in to change notification settings - Fork 21
Conversation
9a0cd3a
to
d0d3610
Compare
crossref Eyescale/Lunchbox#294 |
@@ -59,6 +60,7 @@ int main( const int argc, char** argv ) | |||
{ | |||
if( std::string( "--help" ) == argv[i] ) | |||
{ | |||
livre::DataSource::loadPlugins(); // needed to complete --volume |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
hide in Client::getHelp(). Or leave it, as it will conflict with my upcoming changes
@@ -20,64 +20,33 @@ | |||
|
|||
#include <livre/core/defines.h> | |||
#include <livre/core/configuration/Configuration.h> | |||
#include <lunchbox/term.h> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
missing .gitsubprojects bump then
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Missing +1 in Eyescale/Lunchbox#294
@@ -47,6 +47,8 @@ class DataSource | |||
/** Load all plugin DSOs. */ | |||
LIVRECORE_API static void loadPlugins(); | |||
|
|||
static std::string getDescriptions(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
doxygen?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
renderSettings.setID( servus::make_uint128( "eq::RenderSettings" )); | ||
cameraSettings.setID( servus::make_uint128( "eq::CameraSettings" )); | ||
volumeSettings.setID( servus::make_uint128( "eq::VolumeSettings" )); | ||
frameSettings.setID( servus::make_uint128( "livre::FrameSettings" )); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
missing rebase?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changelog: Remove config file support, new help printing with current terminal size layout
Changelog done |
Retest this please |
No description provided.