Yedidyah Bar David has posted comments on this change.

Change subject: packaging: setup: text changes
......................................................................


Patch Set 1:

(4 comments)

....................................................
File packaging/setup/ovirt_engine_setup/constants.py
Line 939: 
Line 940:     @osetupattrs(
Line 941:         answerfile=True,
Line 942:         summary=True,
Line 943:         description=_('Set application as root page'),
Isn't that too long? It's for the "configuration preview" at the end of 
customization.
Line 944:         postinstallfile=True,
Line 945:     )
Line 946:     def CONFIGURE_ROOT_REDIRECTION(self):
Line 947:         return 'OVESETUP_APACHE/configureRootRedirection'


....................................................
File packaging/setup/plugins/ovirt-engine-setup/apache/root.py
Line 82:             osetupcons.ApacheEnv.CONFIGURE_ROOT_REDIRECTION
Line 83:         ] is None:
Line 84:             self.dialog.note(
Line 85:                 _(
Line 86:                     'Setup can configure the root page of the '
Already changed to 'default' by Alon's suggestion. 'domain root'  does not 
sound good to me, but I am not an English native...
Line 87:                     'web server to present the ovirt-engine 
application. '
Line 88:                     'This may conflict with existing applications.'
Line 89:                 )
Line 90:             )


Line 83:         ] is None:
Line 84:             self.dialog.note(
Line 85:                 _(
Line 86:                     'Setup can configure the root page of the '
Line 87:                     'web server to present the ovirt-engine 
application. '
Done, although it's not clear which application IMO
Line 88:                     'This may conflict with existing applications.'
Line 89:                 )
Line 90:             )
Line 91:             self.environment[


Line 93:             ] = dialog.queryBoolean(
Line 94:                 dialog=self.dialog,
Line 95:                 name='OVESETUP_APACHE_CONFIG_ROOT_REDIRECTION',
Line 96:                 note=_(
Line 97:                     'Do you wish to set ovirt-engine as the root page 
of '
same...

I think I'll quit now and save my attitude to when we release hebrew 
localization :-)
Line 98:                     'the web server? (@VALUES@) [@DEFAULT@]: '
Line 99:                 ),
Line 100:                 prompt=True,
Line 101:                 default=self.environment[


-- 
To view, visit http://gerrit.ovirt.org/18053
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ic2a50fdb2e3b2281fafa894b8e95c7902ba24380
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Yedidyah Bar David <d...@redhat.com>
Gerrit-Reviewer: Alon Bar-Lev <alo...@redhat.com>
Gerrit-Reviewer: Ofer Schreiber <oschr...@redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbona...@redhat.com>
Gerrit-Reviewer: Yedidyah Bar David <d...@redhat.com>
Gerrit-HasComments: Yes
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to