Age | Commit message (Collapse) | Author |
|
The change to use backticks to capture the output was already done for
the main package in 6c4cb4e7e0afce2989c9fe596fd74a35eece6206, this
change now also fixes it for the langpacks
Change-Id: I26dcfb56342b9c3fc7b9c93f7d586c5c899b7232
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/115808
Tested-by: Jenkins
Reviewed-by: Christian Lohmaier <lohmaier+LibreOffice@googlemail.com>
|
|
OOODOWNLOADNAME was set in the global section of openoffice.lst, so the
code always did set the target path with set_download_filename. Thus
remove the superfluous sub and then also the no longer used
OOODOWNLOADNAME variables
In the same vein: the "script that extracts a tarball that is appended
to it" method is no longer in use since many years, delete that as well.
Change-Id: I43481a3ec09e064ef77138e1cbfc1dba6854f2d5
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/110017
Tested-by: Jenkins
Reviewed-by: Christian Lohmaier <lohmaier+LibreOffice@googlemail.com>
|
|
the scpactions are what assemble the dmg (add the license & readme
folders, the background image and the .DS_Store for alignment/the
layout).
Adding the AARCH64 statement is necessary to add the architecture to the
filename.
Change-Id: I0095b8d631cdaa7ce4333fde15dd17d4faf44e8f
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/109818
Tested-by: Jenkins
Reviewed-by: Christian Lohmaier <lohmaier+LibreOffice@googlemail.com>
|
|
by replacing system() call with backticks operator, since now the
macosx-codesign-app-bundle script doesn't redirect the output to
temp-files anymore (see e2fcbac50549ca96b092d64bd14a37cee6b12e0a)
That way the output can be added to the log in case of failure.
Change-Id: I954cbb2ed28cfe0f7e1e8ebc8f21aeb1711a4d9a
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/109816
Tested-by: Jenkins
Reviewed-by: Christian Lohmaier <lohmaier+LibreOffice@googlemail.com>
|
|
Signing them as executable code would require external attributes, and
those in turn break packaging into hfs+ dmg when building on apfs with
Big Sur.
It is not a new thing - the old Code Signing in Depth technote
https://developer.apple.com/library/archive/technotes/tn2206/_index.html
already reads:
"Store Python, Perl, shell, and other script files and other non-Mach-O
executables in your app's Contents/Resources directory. While it's
possible to sign such executables and store them in Contents/MacOS, this
is not recommended.
[…]
Put another way, a properly-signed app that has all of its files in the
correct places will not contain any signatures stored as extended
attributes."
The patch does exactly that for LO and the shipped python framework and
adds symlinks for the moved files.
Same applies for the Language pack applescript and the tarball - those
are also moved into Contents/Resources
Change-Id: Iab21e77b73f941248ca89c6e80703fdf67a1057c
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/109537
Tested-by: Jenkins
Reviewed-by: Christian Lohmaier <lohmaier+LibreOffice@googlemail.com>
|
|
e.g. Instead of "Base module for LibreOffice 7.0 .3.1" we will get
"Base module for LibreOffice 7.0.3.1".
Change-Id: I9ce56b33f835a9527f777cfb9c0e466232e6f24f
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/105449
Tested-by: Jenkins
Reviewed-by: Christian Lohmaier <lohmaier+LibreOffice@googlemail.com>
|
|
Change-Id: I31f1b1aae3b89a58c33f58a76e32bf171f344012
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/103293
Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice@gmail.com>
Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
(cherry picked from commit 7e610406a2d063ba5efacb0e186524c2a4ed77a9)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/103203
Tested-by: Jenkins
|
|
Like these:
Use of uninitialized value $feature{"Description"} in substitution (s///) at C:/lo/src/core/solenv/bin/modules/installer/windows/feature.pm line 362.
Use of uninitialized value $feature{"Title"} in concatenation (.) or string at C:/lo/src/core/solenv/bin/modules/installer/windows/feature.pm line 370.
Use of uninitialized value $feature{"Description"} in concatenation (.) or string at C:/lo/src/core/solenv/bin/modules/installer/windows/feature.pm line 370.
gm_o_OGLTrans has no Title nor Description
Change-Id: Ica247e071f252023e4cdec6f8ff15f0e0d3cedd5
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/102059
Tested-by: Jenkins
Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
|
|
Use of uninitialized value $microversion in int at C:/lo/src/core/solenv/bin/modules/installer/windows/file.pm line 568.
Use of uninitialized value $vervariant in int at C:/lo/src/core/solenv/bin/modules/installer/windows/file.pm line 568.
Change-Id: I916861a34d39a796d2941f9a8d972e7a082e3b22
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/102058
Tested-by: Jenkins
Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
|
|
Like these:
Use of uninitialized value in exists at C:/lo/src/core/solenv/bin/modules/installer/scriptitems.pm line 402.
Debugging the failing values, these are:
element 113
Dir => gid_Dir_Autotext_Language
HostName => LibreOfficeDev 7/share/autotext/qtz
modules => gid_Module_Langpack_Basis_qtz
specificlanguage => qtz
Styles => (CREATE)
element 512
Dir => gid_Brand_Dir_Share_Uno_Packages
HostName => LibreOfficeDev 7/share/uno_packages
modules => gid_Module_Root,gid_Module_Root_Brand
specificlanguage =>
Styles => (CREATE)
element 513
Dir => gid_Brand_Dir_Share_Uno_Packages_Cache
HostName => LibreOfficeDev 7/share/uno_packages/cache
modules => gid_Module_Root,gid_Module_Root_Brand
specificlanguage =>
Styles => (CREATE)
element 514
Dir => gid_Brand_Dir_Share_Uno_Packages_Cache_Uno_Packages
HostName => LibreOfficeDev 7/share/uno_packages/cache/uno_packages
modules => gid_Module_Root
specificlanguage =>
Styles => (CREATE)
Change-Id: I9c91e2fa9bbaab4c15e96dbbdc710141e500c482
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/102053
Tested-by: Jenkins
Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
|
|
This version was introduced in b173b425a1c1478ab3c4b43e58e09b751e3e3387,
then in cabadfc288e5e7324723c62f36b918a80db90323, another copy was added
so that now install.phony.log had this line:
Subroutine is_empty_dir redefined at C:/lo/src/core/solenv/bin/modules/installer/systemactions.pm line 475.
Let's keep the newer version, that is at line 280.
Change-Id: I50f3668e47d4ea8f9c309abde8abe233cff12af5
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/102046
Tested-by: Jenkins
Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
|
|
Redist is a system component, that includes a varying set of DLLs,
and those DLLs are ref-counted. Installing a newer redist - i.e.
updating and increasing refcount of existing DLLs - may add new
DLLs (with initial refcount 1) in addition to the updated old DLLs
that start depending on the newly added ones; at uninstall, the
newly added DLLs may get removed because their refcount gets 0,
while other redist DLLs are kept at the updated levels - so their
dependencies now are not met, and redist gets broken.
Just mark the redist components permanent, which, according to [1],
"registers an extra system client for the component in the Windows
Installer registry settings".
A downside is that uninstall doesn't restore the original system
state ideally.
[1] https://docs.microsoft.com/en-us/windows/win32/msi/component-table
Change-Id: I3fe82bcb5844f826f5b1df622273b4e3a1e3c436
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/100426
Tested-by: Mike Kaganski <mike.kaganski@collabora.com>
Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
|
|
Noop. Use --with-extra-buildid
Change-Id: I110514f6f58c1cd2ca635234a20cf560e6e3d6e6
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/98923
Tested-by: Jenkins
Reviewed-by: Miklos Vajna <vmiklos@collabora.com>
|
|
Makes it easier to see which is which in Finder if you have several
mounted.
Change-Id: I6e1f5cfc4db3ed6595ba14e515080b5ecc0ad688
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/94120
Tested-by: Jenkins
Reviewed-by: Tor Lillqvist <tml@collabora.com>
|
|
Change-Id: I709561ba910ff2274a40e578412ecd7040886959
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/94144
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
|
|
(Not something that is used when actually building for distribution,
sure, but when testing.)
Change-Id: I67410a8cd9627c5a2c19f58df4529a44e488c961
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/94119
Tested-by: Jenkins
Reviewed-by: Tor Lillqvist <tml@collabora.com>
|
|
Change-Id: Iead0a7e770a5533e3fe4585f76e343f0039328fd
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/94105
Tested-by: Jenkins
Reviewed-by: Tor Lillqvist <tml@collabora.com>
|
|
88453fd13a653963b394b8f865ff104b5545f137 changed the
hardened_runtime.xcent to a generated file
Change-Id: I54b927d1990835bb902f62e03d51b0e70735269c
|
|
'make test-install -o build' fails without this when
--enable-macosx-sandbox is used. It is harmless in other cases.
Change-Id: Ic62a2c7729402cf45172ccc12fa83b46bee31e78
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/92985
Tested-by: Jenkins
Reviewed-by: Miklos Vajna <vmiklos@collabora.com>
|
|
Ende -> End
Change-Id: I2d63ddfeb1fa59ed1b0e3bbf2d39d5221cd5425f
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/92026
Tested-by: Jenkins
Reviewed-by: Michael Weghorn <m.weghorn@posteo.de>
|
|
macOS 10.14 and later (Mojave & Catalina) will treat signed exectuables
as "restricted binary" and in turn will block any use of
DYLD_LIBRARY_PATH settings, causing the build to fail with e.g.:
dyld: Library not loaded: @__VIA_LIBRARY_PATH__/libuno_sal.dylib.3
Referenced from: /Users/cloph/build/workdir/LinkTarget/Executable/genconv_dict
Reason: unsafe use of relative rpath @__VIA_LIBRARY_PATH__/libuno_sal.dylib.3 in /Users/cloph/build/workdir/LinkTarget/Executable/genconv_dict with restricted binary
Change-Id: Ia90d8fa6f28f9a2c10f6b46eb1796e2730550119
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/81599
Tested-by: Jenkins
Reviewed-by: Christian Lohmaier <lohmaier+LibreOffice@googlemail.com>
|
|
This registers SharePoint integration libraries using regsvr.exe.
Both 32-bit and 64-bit libraries are registered; registration of
LOSPSupport.OpenDocuments is unconditional.
This introduces a new hidden MSI feature, which is disabled for
installation: gm_SharePointSupport_SubstMSO. When installed, it
registers SharePoint.OpenDocuments class in registry, thus
overriding registration of this component by MS Office, allowing
LibreOffice to serve as MS Office replacement working in IE with
SharePoint. To install the feature, either a transform is needed
setting the feature's level <= 100, or a command line:
msiexec path-to-msi ADDLOCAL=gm_SharePointSupport_SubstMSO
Change-Id: I5517bbb68dcc6db8bcb2bbc2368394ee4a62d741
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/86452
Tested-by: Jenkins
Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
|
|
...which had been introduced, already unused, with
1e4422a4b4c443c286c90853f50b739a67ce5d1f "re-base on ALv2 code. Includes:"
Change-Id: I5a26d7365c510e861bd25e0882cfe7292695107c
Reviewed-on: https://gerrit.libreoffice.org/82108
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
|
|
(cf. ScpAction scp_Copy_Ds_Store in scp2/source/ooo/scpaction_ooo.scp making use
of this feature)
Dev builds overruling the .DS_Store file for language-/helppacks appears to have
been broken ever since dab79744767c05fd5268866937e225b74e658cea "Give Mac master
builds a nice background .dmg image, too".
Change-Id: I7473d1b0b35fe0b435e7d26c0639e26ed157245d
Reviewed-on: https://gerrit.libreoffice.org/82107
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
|
|
as with private key on crypto-smartcard you'd have to enter your pin
over a hundred times while creating full-lang builds and that is not
fun.
This reduces it to
* once for dll/exe
(at least in case for mergelib is is less than 350 objects and
that doesn't break commandline limits - previously it was set to
only sign 20 objects at a time, forcing a pin-entry over 15 times) and
* once for main installation set
* once for SDK
* once for all the helppacks
(signing description previously also contained the language, this
change drops that to just "<productname> <version> Helppack"
and last three are not scattered timewise, but are done after all
packaging is complete, so the build only waits twice for user-input.
Change-Id: Ibb8bb233e967556f9654573ad30d0ed5883b533f
Reviewed-on: https://gerrit.libreoffice.org/78649
Tested-by: Jenkins
Reviewed-by: Christian Lohmaier <lohmaier+LibreOffice@googlemail.com>
|
|
Change-Id: I3424e17cfdfb563fdc5882942031deafae8689fe
Reviewed-on: https://gerrit.libreoffice.org/78678
Tested-by: Jenkins
Reviewed-by: Julien Nabet <serval2412@yahoo.fr>
|
|
Change-Id: I66d48c809b263835016571772ce2ee529c16b8f4
Reviewed-on: https://gerrit.libreoffice.org/76960
Tested-by: Jenkins
Reviewed-by: Julien Nabet <serval2412@yahoo.fr>
|
|
Fedora 30's rpm 4.14.2.1 will generate this Requires:
config(lodevbasis6.1-core) = 6.1.7.0.0-1
... but no corresponding Provides because that's disabled, so the
package cannot be installed.
The reason is that there is one %config file in the -core rpm:
%attr(0644,root,root) %config(noreplace) "/opt/libreofficedev6.1/./share/psprint/psprint.conf"
Old RPM from CentOS6 does not generate these config requires.
Unfortunately there doesn't appear to be a way to disable this
config(...) without disabling AutoReq, so do that, and invoke the shell
script find-requires-x11.sh manually from epmfile.pm.
Change-Id: I7fee0d9cd1b9e79f81bd4c611500e84736564881
Reviewed-on: https://gerrit.libreoffice.org/76736
Tested-by: Jenkins
Reviewed-by: Michael Stahl <Michael.Stahl@cib.de>
|
|
...where my --with-package-format=dmg build routinely fails with
> : *********************************************************************
> : ERROR: The following errors occurred in packaging process:
> :
> : ERROR: Could not execute "cd /Users/stephan/Software/lo/core/workdir/installation/LibreOffice/dmg/install/en-US_inprogress_temp.19857 && hdiutil create -megabytes 1500 -srcfolder LibreOffice_6.4.0.0.alpha0_MacOS_x86-64 /Users/stephan/Software/lo/core/workdir/installation/LibreOffice/dmg/install/en-US_inprogress/LibreOffice_6.4.0.0.alpha0_MacOS_x86-64.dmg -ov -fs HFS+ -volname "LibreOffice" -format UDBZ": 256
> : *********************************************************************
No idea whether -megabytes 1500 was chosen for a specific reason, so leave it
alone for --disable-dbgutil builds for now.
Change-Id: Ie89d1043e47ed3c84167cbb22325513305013f86
Reviewed-on: https://gerrit.libreoffice.org/75775
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
|
|
...which will be needed in a subsequent patch to fix tdf#125693.
The installer code could not handle a directory in a .filelist, the simplest
hack appears to be to add a special case for an empty source dir to
copy_one_file in solenv/bin/modules/installer/systemactions.pm.
Includes changes made by Stephan Bergmann <sbergman@redhat.com>.
Change-Id: I52dca2543a66eb76117598d77d559592e26ce859
Reviewed-on: https://gerrit.libreoffice.org/75702
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
|
|
At least since 7a08bfeabe21193e04b9747a831653efcfc63190 "tdf#122244 Put
InfoPlist.strings files at correct places on macOS" those directories that have
actual content are created on demand. (That is, empty such directories are no
longer created, but I assume that's a feature rather than a bug.)
Change-Id: If753483c78fbf39e76c9565a407e7ff27ca32efb
Reviewed-on: https://gerrit.libreoffice.org/71300
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
|
|
Had found
> sh: -c: line 0: syntax error near unexpected token `('
> sh: -c: line 0: `file /Users/stephan/Software/lo/core/instdir/LibreOffice.app/Contents/Frameworks/LibreOfficePython.framework/Versions/3.5/lib/python3.5/site-packages/setuptools/script (dev).tmpl'
in workdir/CustomTarget/instsetoo_native/install/install.phony.log on macOS.
Change-Id: I4b01b5f0a292424a60558cdb8b431a3d8dbdf1d4
Reviewed-on: https://gerrit.libreoffice.org/71234
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
|
|
...to make it easier to debug a failing build
Change-Id: Ic936a6e6026db4208f58e19fb27e2c2d6e4adfec
Reviewed-on: https://gerrit.libreoffice.org/70138
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
|
|
This reverts commit c2f121842edcfba060c46343d78d4ef4c28b6249.
Change-Id: Id4c8a0311e93cb5befd61abccfe994750a67df22
|
|
Change-Id: If80ac5648f50deda7a16bc6661ced31357958d67
Reviewed-on: https://gerrit.libreoffice.org/70038
Tested-by: Jenkins
Reviewed-by: Andras Timar <andras.timar@collabora.com>
|
|
... for easier maintenance. This commit only introduces the
framework. Follow up commits will update the license file.
Advantages:
* single source, multiple outputs (html, txt)
* conditional text, i.e. output will not include license
terms of components that are not configured in the build
Change-Id: I0ffad41119228a0a26fd1b6ce19930ca6085adb2
Reviewed-on: https://gerrit.libreoffice.org/69396
Tested-by: Jenkins
Reviewed-by: Andras Timar <andras.timar@collabora.com>
|
|
Reorder the order the filelists are initialized per package.
Optimize speed of find_files_for_package:
We can simply check if the modules entry is the same as last time.
The code uses iterators instead of that C-ish for loops.
Create a hash from the packagemodules to speed up the check if
a module is included in the hash.
Change-Id: I1e8ae394634a527880e08ef8ff333d94d04f49fd
Before: 12s per language/package
After: <1s per language/package
Reviewed-on: https://gerrit.libreoffice.org/68339
Tested-by: Jenkins
Reviewed-by: Tomáš Chvátal <tchvatal@suse.cz>
|
|
commit c3ab3df902b9e2ad363d1eca14da8f9f7f1567bb
Date: Thu Oct 14 09:53:40 2010 +0200
changed the configure.ac to require "Patched by Libreoffice", which was
possibly an error
Change-Id: I6f8e302baeed054f36b54f8bfb6f5cad826ee788
Reviewed-on: https://gerrit.libreoffice.org/67199
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>
|
|
(cherry picked from commit e774041ec07d7f168c41a42b6ffa6aacc816d442)
Change-Id: Ie21b6a4a20199e4c7b7ef891719916db4ffba545
Reviewed-on: https://gerrit.libreoffice.org/67198
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>
|
|
(cherry picked from commit fef416499d3559230d78288661d73fe59838273e)
Change-Id: I0a8f766fb5255628ea58679acca5c6b5fcefe2b1
Reviewed-on: https://gerrit.libreoffice.org/67196
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>
|
|
(cherry picked from commit e50d4c88a194dccc2b3de451503a10903e3ab006)
Change-Id: Ib5fbbd250fbfa9db5f3df36277f81ffa999632d7
Reviewed-on: https://gerrit.libreoffice.org/67195
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>
|
|
It passed "make check" on Linux
Change-Id: Ic8b2adeb949bfc72830667b6928147ebd053d2f0
Reviewed-on: https://gerrit.libreoffice.org/65517
Tested-by: Jenkins
Reviewed-by: Julien Nabet <serval2412@yahoo.fr>
|
|
Change-Id: I651b7f202fa52ff5f5357a11aa72c43eb7dc7f95
Reviewed-on: https://gerrit.libreoffice.org/64102
Tested-by: Jenkins
Reviewed-by: Bartosz Kosiorek <gang65@poczta.onet.pl>
|
|
Change-Id: I0c966f517e337caa340526b48fa62b9910ce0a3e
Reviewed-on: https://gerrit.libreoffice.org/63504
Reviewed-by: Julien Nabet <serval2412@yahoo.fr>
Tested-by: Julien Nabet <serval2412@yahoo.fr>
|
|
Hopefully this fixes the problem, which is being incorrect version written
for the font in MSI (1.0.0.0), and comparing with installed one (1.3), the
font is removed and not reinstalled.
Change-Id: Iee675e2c2b3f29c76e35dcf41bbcce9b6181bf62
Reviewed-on: https://gerrit.libreoffice.org/60548
Tested-by: Jenkins
Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
|
|
Change-Id: I5d99319aa336c02dc0c20ab2bc3a1302840d100d
Reviewed-on: https://gerrit.libreoffice.org/59242
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
|
|
Change-Id: If0d8f4033d9bc20f521d33d732fb349f0df5eeef
Reviewed-on: https://gerrit.libreoffice.org/57822
Reviewed-by: Julien Nabet <serval2412@yahoo.fr>
Tested-by: Jenkins
|
|
Nobody reads these error messages if the build succeeds.
Change-Id: Ib1e52bf188d0da2bc35a0d2d2f79fff409e15977
Reviewed-on: https://gerrit.libreoffice.org/57270
Tested-by: Jenkins
Reviewed-by: Michael Stahl <Michael.Stahl@cib.de>
|
|
No point in creating funny MSIs that don't work.
(Thanks to bubli for the help with perl)
Change-Id: I54aa39ed282e31270de69566656d33b803a73802
Reviewed-on: https://gerrit.libreoffice.org/57266
Tested-by: Jenkins
Reviewed-by: Michael Stahl <Michael.Stahl@cib.de>
|
|
the s/\@// conflicts with jenkins default directory naming for
concurrent builds and is wrong at this place/in this form:
0ce067c1fc334680a7d99e573ae8c2f3e623b645 already changed the format of
the parameter from @listfile to @@listfile, so stripping a single @ at
the front won't help at all.
Furthermore the stripping already was added to the
par2script/parameters.pm earlier, so the worker functions don't need to
deal with transforming commandline arguments.
Change-Id: Ic55b8cda3cd114dfc5cb7ded6d53174cd8283af9
Reviewed-on: https://gerrit.libreoffice.org/55196
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Christian Lohmaier <lohmaier+LibreOffice@googlemail.com>
|