You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@openoffice.apache.org by GitBox <gi...@apache.org> on 2021/06/19 13:33:17 UTC
[GitHub] [openoffice] ardovm opened a new pull request #133: graphite: remove the -Werror option from debug builds
ardovm opened a new pull request #133:
URL: https://github.com/apache/openoffice/pull/133
The flag is set by default by the graphite configure script, but it lets the compilation fail with gcc 7.5
The compilation of the graphite module is consistently failing on openSUSE Leap 15.2 x86_64, when AOO is being built with debug options enabled. This patch allows successful compilation of the graphite module on such systems.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@openoffice.apache.org
For additional commands, e-mail: dev-help@openoffice.apache.org
[GitHub] [openoffice] leginee commented on pull request #133: graphite: remove the -Werror option from debug builds
Posted by GitBox <gi...@apache.org>.
leginee commented on pull request #133:
URL: https://github.com/apache/openoffice/pull/133#issuecomment-864492889
The meaing of the Flag is
-Werror: every warning is treated as an error.
Since Grafite is a dependency it is fine to build without. However Imho we should strife for removing Warnings from Code.
Maybe it is worth trying to build with a new Version of Grafite on Suse Machines?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@openoffice.apache.org
For additional commands, e-mail: dev-help@openoffice.apache.org
[GitHub] [openoffice] leginee merged pull request #133: graphite: remove the -Werror option from debug builds
Posted by GitBox <gi...@apache.org>.
leginee merged pull request #133:
URL: https://github.com/apache/openoffice/pull/133
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscribe@openoffice.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@openoffice.apache.org
For additional commands, e-mail: dev-help@openoffice.apache.org
[GitHub] [openoffice] leginee commented on pull request #133: graphite: remove the -Werror option from debug builds
Posted by GitBox <gi...@apache.org>.
leginee commented on pull request #133:
URL: https://github.com/apache/openoffice/pull/133#issuecomment-864513194
Then we have 2 choices. Fix the Issues or prepare the switch to a replacement.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@openoffice.apache.org
For additional commands, e-mail: dev-help@openoffice.apache.org
Re: [GitHub] [openoffice] ardovm edited a comment on pull request
#133: graphite: remove the -Werror option from debug builds
Posted by Don Lewis <tr...@apache.org>.
On 20 Jun, Pedro Lino wrote:
> Hi Arrigo, all
>
> Development seems to be quite live at
> https://github.com/silnrsi/graphite
We are using SIL Graphite, http://silgraphite.sourceforge.net/
Looking at the Graphite technical overview page mentioned here:
https://github.com/silnrsi/graphite/blob/master/README.md
(https://scripts.sil.org/cms/scripts/page.php?site_id=projects&item_id=graphite_techAbout)
under "Graphite engine implementations":
There are two existing implementations of the Graphite engine. The
original engine is packaged under the name "SilGraphite"; the initial
implementation was done in 2000. The Graphite2 engine implementation
began in 2010.
The SilGraphite engine is currently used by OpenOffice, XeTeX,
FieldWorks, and WorldPad. It includes some features not included in
Graphite2, but is considerably slower. Graphite2 represents a
significant (10x) optimization in terms of speed and greatly improved
robustness, but does not include a few features such a ligature
component handling and cross-line-boundary contextualization.
Graphite2 is used by LibreOffice and Firefox.
The two engines support different APIs. The Graphite2 API was
specifically developed to be more compatible with the approach to text
layout used by most commercial software. See the application
development page for API documentation.
Graphite2 would be better, is currently supported, and Libreoffice
switched to it quite a while ago, but it is not a drop-in replacement.
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@openoffice.apache.org
For additional commands, e-mail: dev-help@openoffice.apache.org
Re: [GitHub] [openoffice] ardovm edited a comment on pull request
#133: graphite: remove the -Werror option from debug builds
Posted by Pedro Lino <pe...@mailbox.org.INVALID>.
Hi Arrigo, all
Development seems to be quite live at
https://github.com/silnrsi/graphite
Regards,
Pedro
> On 06/20/2021 9:08 PM GitBox <gi...@apache.org> wrote:
>
>
> ardovm edited a comment on pull request #133:
> URL: https://github.com/apache/openoffice/pull/133#issuecomment-864605285
>
>
> The first two warnings are IMHO bugs. The first one is a function parameter type `bool` instead of `bool&` and the second is a mixup of return values in a `switch` statement.
>
> Others are linked to assertions not being compiled (although we are requesting a debug build). Some others are unused variables that are explicitly marked `TODO` in the code.
>
> I stopped after checking the first four warnings.
>
> I suggest we allow this PR through so that at least the source becomes compilable again. Then we could have a longer term plan for the replacement of the graphite module with... whatever is better today.
>
> For what it's worth, both the [SourceForge project page](http://silgraphite.sourceforge.net) and [www.sil.org](https://software.sil.org/products/#supported) refer to http://graphite.sil.org that does not seem to exist.
>
>
> --
> This is an automated message from the Apache Git Service.
> To respond to the message, please log on to GitHub and use the
> URL above to go to the specific comment.
>
> For queries about this service, please contact Infrastructure at:
> users@infra.apache.org
>
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@openoffice.apache.org
> For additional commands, e-mail: dev-help@openoffice.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@openoffice.apache.org
For additional commands, e-mail: dev-help@openoffice.apache.org
[GitHub] [openoffice] ardovm edited a comment on pull request #133: graphite: remove the -Werror option from debug builds
Posted by GitBox <gi...@apache.org>.
ardovm edited a comment on pull request #133:
URL: https://github.com/apache/openoffice/pull/133#issuecomment-864605285
The first two warnings are IMHO bugs. The first one is a function parameter type `bool` instead of `bool&` and the second is a mixup of return values in a `switch` statement.
Others are linked to assertions not being compiled (although we are requesting a debug build). Some others are unused variables that are explicitly marked `TODO` in the code.
I stopped after checking the first four warnings.
I suggest we allow this PR through so that at least the source becomes compilable again. Then we could have a longer term plan for the replacement of the graphite module with... whatever is better today.
For what it's worth, both the [SourceForge project page](http://silgraphite.sourceforge.net) and [www.sil.org](https://software.sil.org/products/#supported) refer to http://graphite.sil.org that does not seem to exist.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@openoffice.apache.org
For additional commands, e-mail: dev-help@openoffice.apache.org
[GitHub] [openoffice] leginee commented on pull request #133: graphite: remove the -Werror option from debug builds
Posted by GitBox <gi...@apache.org>.
leginee commented on pull request #133:
URL: https://github.com/apache/openoffice/pull/133#issuecomment-873534871
Okay, lets merge. The issues are there anyhow. I dont think the code is better or worse with the change.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscribe@openoffice.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@openoffice.apache.org
For additional commands, e-mail: dev-help@openoffice.apache.org
[GitHub] [openoffice] ardovm commented on pull request #133: graphite: remove the -Werror option from debug builds
Posted by GitBox <gi...@apache.org>.
ardovm commented on pull request #133:
URL: https://github.com/apache/openoffice/pull/133#issuecomment-864605285
The first two warnings are IMHO bugs. The first one is a function parameter type `bool` instead of `bool&` and the second is a mixup of return values in a `switch` statement.
Others are linked to assertions not being compiled (although we are requesting a debug build). Some others are unused variables that are explicitly marked `TODO` in the code.
I stopped after checking the first four warnings.
I suggest we allow this PR through so that at least the source becomes compilable again. Then we could have a longer term plan for the replacement of the graphite with... whatever is better today.
For what it's worth, both the [SourceForge project page](http://silgraphite.sourceforge.net) and [www.sil.org](https://software.sil.org/products/#supported) refer to http://graphite.sil.org that does not seem to exist.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@openoffice.apache.org
For additional commands, e-mail: dev-help@openoffice.apache.org
[GitHub] [openoffice] DonLewisFreeBSD commented on pull request #133: graphite: remove the -Werror option from debug builds
Posted by GitBox <gi...@apache.org>.
DonLewisFreeBSD commented on pull request #133:
URL: https://github.com/apache/openoffice/pull/133#issuecomment-864508440
> The meaing of the Flag is
> -Werror: every warning is treated as an error.
>
> Since Grafite is a dependency it is fine to build without. However Imho we should strife for removing Warnings from Code.
Yes. It could be an indication of an actual bug.
> Maybe it is worth trying to build with a new Version of Grafite on Suse Machines?
It is long dead upstream. The replacement has an incompatible API.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@openoffice.apache.org
For additional commands, e-mail: dev-help@openoffice.apache.org