[Libreoffice-bugs] [Bug 158108] Build failure with ICU 74

[bugzilla-daemon]

https://bugs.documentfoundation.org/show_bug.cgi?id=158108

--- Comment #6 from taichi <20001...@ymail.ne.jp> ---
(In reply to Eike Rathke from comment #5)
> Well, fine, but what does that actually _mean_ for our line break rules?
> 
> Adding Khaled to Cc, maybe he knows as he touched
> i18npool/source/breakiterator/data/line.txt recently.
> 
> IMHO it's now time to align our rules with ICU upstream's
> https://github.com/unicode-org/icu/blob/main/icu4c/source/data/brkitr/rules/
> line.txt and reapply  the historically grown changes _if still necessary_.

The minimum version of ICU required to build LibreOffice is 66.
Is it no problem to use icu4c/source/data/brkitr/rules/line.txt from 74.1 or
the main branch?

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 158108] Build failure with ICU 74

[bugzilla-daemon]

https://bugs.documentfoundation.org/show_bug.cgi?id=158108

BogdanB  changed:

   What|Removed |Added

 Blocks||149092
 CC||buzea.bog...@libreoffice.or
   ||g


Referenced Bugs:

https://bugs.documentfoundation.org/show_bug.cgi?id=149092
[Bug 149092] [META] Bugs and enhancements related to ICU
-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 158108] Build failure with ICU 74

[bugzilla-daemon]

https://bugs.documentfoundation.org/show_bug.cgi?id=158108

--- Comment #5 from Eike Rathke  ---
Well, fine, but what does that actually _mean_ for our line break rules?

Adding Khaled to Cc, maybe he knows as he touched
i18npool/source/breakiterator/data/line.txt recently.

IMHO it's now time to align our rules with ICU upstream's
https://github.com/unicode-org/icu/blob/main/icu4c/source/data/brkitr/rules/line.txt
and reapply  the historically grown changes _if still necessary_.

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 158108] Build failure with ICU 74

[bugzilla-daemon]

https://bugs.documentfoundation.org/show_bug.cgi?id=158108

--- Comment #4 from taichi <20001...@ymail.ne.jp> ---
(In reply to Eike Rathke from comment #3)
> (In reply to Natanael Copa from comment #0)
> > This appears to have been removed in ICU in this commit:
> > https://github.com/unicode-org/icu/commit/
> > 84e47620692be90950d090f2f4722494b020ad96
> Oh great, "Within ICU, it is used only with the line break
> rules. We hope to replace it with something more general." They hoped. But
> apparently they didn't?
> 
> However, I suggest to simply remove that one line, or maybe better turn it
> into a comment and point to this bug here / the ICU commit there.


I found a webpage related to this bug.

> The !!LBCMNoChain option (Line Break Combining Marks No Chain) will be
> disappearing. It was a hack used in implementing the Unicode line break rules,
> and with the new !^ pattern character providing finer grained control over
> rule chaining, it is no longer necessary.

"[ICU-12331] Update UserGuide for new BreakIterator behavior and rule syntax -
Unicode Consortium"


-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 158108] Build failure with ICU 74

[bugzilla-daemon]

https://bugs.documentfoundation.org/show_bug.cgi?id=158108

--- Comment #3 from Eike Rathke  ---
(In reply to Natanael Copa from comment #0)
> This appears to have been removed in ICU in this commit:
> https://github.com/unicode-org/icu/commit/
> 84e47620692be90950d090f2f4722494b020ad96
Oh great, "Within ICU, it is used only with the line break
rules. We hope to replace it with something more general." They hoped. But
apparently they didn't?

However, I suggest to simply remove that one line, or maybe better turn it into
a comment and point to this bug here / the ICU commit there.

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 158108] Build failure with ICU 74

[bugzilla-daemon]

https://bugs.documentfoundation.org/show_bug.cgi?id=158108

--- Comment #2 from Natanael Copa  ---
(In reply to Julien Nabet from comment #1)
> There's a patch which needs some tweaks here:
> https://gerrit.libreoffice.org/c/core/+/158749

The comment there shows that they bumped into same genbrk issue as I did when
building with system ICU.

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 158108] Build failure with ICU 74

[bugzilla-daemon]

https://bugs.documentfoundation.org/show_bug.cgi?id=158108

Julien Nabet  changed:

   What|Removed |Added

 CC||20001...@ymail.ne.jp,
   ||er...@redhat.com,
   ||serval2...@yahoo.fr
 Ever confirmed|0   |1
 Status|UNCONFIRMED |NEW

--- Comment #1 from Julien Nabet  ---
There's a patch which needs some tweaks here:
https://gerrit.libreoffice.org/c/core/+/158749

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 158108] Build failure with ICU 74

[bugzilla-daemon]

https://bugs.documentfoundation.org/show_bug.cgi?id=158108

Robert Großkopf  changed:

   What|Removed |Added

  Component|Base|LibreOffice

-- 
You are receiving this mail because:
You are the assignee for the bug.