Bug 124456 - Make Ant configue.ac check more robust
Summary: Make Ant configue.ac check more robust
Status: NEW
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: LibreOffice (show other bugs)
Version:
(earliest affected)
6.2.2.2 release
Hardware: All All
: medium normal
Assignee: Not Assigned
URL:
Whiteboard:
Keywords: patch
Depends on:
Blocks:
 
Reported: 2019-03-31 18:07 UTC by Dennis Schridde
Modified: 2019-07-09 14:22 UTC (History)
3 users (show)

See Also:
Crash report or crash signature:


Attachments
libreoffice-6.2.2.2-stricter-ant-check.patch (584 bytes, patch)
2019-03-31 18:08 UTC, Dennis Schridde
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Dennis Schridde 2019-03-31 18:07:58 UTC
Description:
Attached patch makes the configure.ac check for Ant more robust by specifically searching for the line containing the version information.

See-Also: https://bugs.gentoo.org/682156
See-Also: https://bugs.gentoo.org/682136

Steps to Reproduce:
At least on Gentoo, Ant is wrapped in a shell script that might output additional lines.  This would lead to the build of LibreOffice 6.2.2.2 failing with following message: checking whether Ant is >= 1.6.0... ./configure: line 39234: test: determine because are have 1: integer expression expected

Actual Results:
checking whether Ant is >= 1.6.0... ./configure: line 39234: test: determine because are have 1: integer expression expected

Expected Results:
LibreOffice's configure.ac should ignore any extraneous lines in the output of `ant -version` and focus on the line containing the version information.


Reproducible: Always


User Profile Reset: No



Additional Info:
Comment 1 Dennis Schridde 2019-03-31 18:08:25 UTC
Created attachment 150430 [details]
libreoffice-6.2.2.2-stricter-ant-check.patch
Comment 2 Dennis Schridde 2019-04-23 07:05:55 UTC
Issue persists with LibreOffice 6.2.3.2.
Comment 3 Xisco Faulí 2019-04-23 17:17:25 UTC
Hello Dennis,
Thanks for providing the patch.
Could you please submit it to gerrit for review ? -> https://wiki.documentfoundation.org/Development/gerrit/SubmitPatch
Comment 4 Xisco Faulí 2019-07-09 14:22:38 UTC
(In reply to Xisco Faulí from comment #3)
> Hello Dennis,
> Thanks for providing the patch.
> Could you please submit it to gerrit for review ? ->
> https://wiki.documentfoundation.org/Development/gerrit/SubmitPatch

Setting to NEEDINFO until Dennis(In reply to Xisco Faulí from comment #3)
> Hello Dennis,
> Thanks for providing the patch.
> Could you please submit it to gerrit for review ? ->
> https://wiki.documentfoundation.org/Development/gerrit/SubmitPatch

Moving to NEW
@Dennis, Please submit the patch to gerrit