Merge lp:~googol-deactivatedaccount/openlp/bug-902964-2 into lp:openlp/2.0

Proposed by Andreas Preikschat
Status: Rejected
Rejected by: Andreas Preikschat
Proposed branch: lp:~googol-deactivatedaccount/openlp/bug-902964-2
Merge into: lp:openlp/2.0
Diff against target: 25 lines (+4/-4)
1 file modified
openlp/core/utils/__init__.py (+4/-4)
To merge this branch: bzr merge lp:~googol-deactivatedaccount/openlp/bug-902964-2
Reviewer Review Type Date Requested Status
Raoul Snyman Approve
Review via email: mp+151088@code.launchpad.net

This proposal supersedes a proposal from 2013-02-28.

Description of the change

Hello,

- fixed bug #902964 (Libreoffice deprecated commandline args)

To post a comment you must log in.
Revision history for this message
Raoul Snyman (raoul-snyman) wrote :

This *should* be OK on modern systems, but on older systems like Ubuntu 10.04 things will fail. Do we want to up our minimum versions again?

review: Approve
Revision history for this message
Jonathan Corwin (j-corwin) wrote :

My view is that if someone is running an older system, then they should also be happy running an older OpenLP (i.e. 2.0).
If at some point they really want/need to run a newer OpenLP then they ought to be prepared to upgrade the system too.

Revision history for this message
Raoul Snyman (raoul-snyman) wrote :

Jonathan, that's fine, but this proposal is meant to go into 2.0 ;-)

Revision history for this message
Jonathan Corwin (j-corwin) wrote :

Ahh, I hadn't spotted that.
Why do we need to fix this in 2.0, aren't they just warning messages?

Revision history for this message
Andreas Preikschat (googol-deactivatedaccount) wrote :

We can skip this proposal for the 2.0 branch....

Unmerged revisions

2138. By Andreas Preikschat

fixed mistake

2137. By Andreas Preikschat

- fixed bug #902964 (Libreoffice deprecated commandline args)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'openlp/core/utils/__init__.py'
2--- openlp/core/utils/__init__.py 2012-12-30 19:41:24 +0000
3+++ openlp/core/utils/__init__.py 2013-02-28 20:19:23 +0000
4@@ -91,7 +91,7 @@
5 VersionDir = 5
6 CacheDir = 6
7 LanguageDir = 7
8-
9+
10 # Base path where data/config/cache dir is located
11 BaseDir = None
12
13@@ -447,11 +447,11 @@
14 Returns the UNO command to launch an openoffice.org instance.
15 """
16 COMMAND = u'soffice'
17- OPTIONS = u'-nologo -norestore -minimized -nodefault -nofirststartwizard'
18+ OPTIONS = u'--nologo --norestore --minimized --nodefault --nofirststartwizard'
19 if UNO_CONNECTION_TYPE == u'pipe':
20- CONNECTION = u'"-accept=pipe,name=openlp_pipe;urp;"'
21+ CONNECTION = u'"--accept=pipe,name=openlp_pipe;urp;"'
22 else:
23- CONNECTION = u'"-accept=socket,host=localhost,port=2002;urp;"'
24+ CONNECTION = u'"--accept=socket,host=localhost,port=2002;urp;"'
25 return u'%s %s %s' % (COMMAND, OPTIONS, CONNECTION)
26
27

Subscribers

People subscribed via source and target branches