Issue 125501 - Spellcheck does not work, category B modules are missing (OS/2)
Summary: Spellcheck does not work, category B modules are missing (OS/2)
Status: CLOSED FIXED
Alias: None
Product: General
Classification: Code
Component: spell checking (show other issues)
Version: 4.1.0
Hardware: PC OS/2
: P3 Major (vote)
Target Milestone: 4.1.2
Assignee: Yuri Dario
QA Contact:
URL:
Keywords:
Depends on:
Blocks: 118923
  Show dependency tree
 
Reported: 2014-08-25 07:03 UTC by Lewis Rosenthal
Modified: 2016-12-31 15:37 UTC (History)
5 users (show)

See Also:
Issue Type: DEFECT
Latest Confirmation in: 4.1.0
Developer Difficulty: ---
pescetti: 4.1.2_release_blocker+


Attachments

Note You need to log in before you can comment on or make changes to this issue.
Description Lewis Rosenthal 2014-08-25 07:03:27 UTC
This may be invalid, as I am reporting this for a release candidate, but here it is, anyway:

Installed fresh 4.1.1 RC3, with new profile. Downloaded and installed latest English dictionary from official repository (compatible with OO4). Enabled option to check spelling while typing; no reported errors (even when intentionally misspelling). Pressing F7 or the toolbar button to begin a spellcheck finishes instantly with a message that no errors have been detected. Clicking OK closes the box (cannot stay to adjust settings).

In options, I do not see where I might have missed enabling anything.
Comment 1 oooforum (fr) 2014-08-25 08:55:05 UTC
Dup of issue 121930
Comment 2 Lewis Rosenthal 2014-08-25 15:28:14 UTC
This is not a dup of issue 121930.

Deleting the profile does not resolve this. This is apparently unique to the OS/2 RC.
Comment 3 kbr 2014-08-25 19:29:17 UTC
I can confirm this. Deleting the user profile does not make spellchecking work.
Comment 4 Yuri Dario 2014-08-26 13:22:35 UTC
Same issue here.
Comment 5 Yuri Dario 2014-08-28 10:34:47 UTC
I first enabled spell modules, but then it turned out that it is part of so-called Category B items.
So I have enabled Category B, this made spellchecker to be included; also solver module is now working, plus xslt filters and more scripting tools.
Comment 6 Yuri Dario 2014-08-28 11:48:15 UTC
#i125501# build fixes for enabling Category B also in OS/2 port.
Committed revision 1621121.
Comment 7 Yuri Dario 2014-08-28 12:00:30 UTC
A binary update has been uploaded to dropbox download directory:

Apache_OpenOffice_4.1.1_os2gcci_install-arc_en-US-RC3-update1.zip

will spellchecker and solver dlls.
Comment 8 Lewis Rosenthal 2014-08-28 14:37:09 UTC
Thanks, Yuri. I haven't tested solver, xslt filters, or scripting, but can confirm that spelling is working (both spell-as-you-type and via the Spelling & grammar dialog).

I'll mark this as fixed, as this was the sole purpose of this bug. Suggest that any other issues which arise now that this is functional raise separate tickets.
Comment 9 Yuri Dario 2014-09-09 15:30:27 UTC
Forgot libs.mk changes...

Committed revision 1623847.
Comment 10 SVN Robot 2014-10-07 09:44:13 UTC
"ydario" committed SVN revision 1623847 into trunk:
#i125501# build fixes for enabling Category B also in OS/2 port.
Comment 11 SVN Robot 2014-10-07 09:47:00 UTC
"ydario" committed SVN revision 1621121 into trunk:
#i125501# build fixes for enabling Category B also in OS/2 port.
Comment 12 SVN Robot 2014-11-28 13:53:52 UTC
"ydario" committed SVN revision 1642300 into trunk:
#i125501# build fixes for enabling Category B also in OS/2 port.
Comment 13 Kay 2015-09-07 21:37:54 UTC
Do these changed need to be ported to AOO410 branch for OS/2 4.1.2 building?
Comment 14 Yuri Dario 2015-09-08 07:16:28 UTC
Kay, how to make sure this code and many others are landing in 4.1.2 branch too? I committed my patches only to trunk because it was not clear how/if to apply them to branches.
Comment 15 Andrea Pescetti 2015-09-10 13:03:45 UTC
It is OK to make this a blocker since OS/2 is not officially distributed, but still it is good to be able to build on as many systems as possible. Code changes affect the OS/2 build only.

@Yuri: feel free to merge your 3 patches to the AOO410 branch, see
https://bz.apache.org/ooo/show_bug.cgi?id=126281#c11
for procedural details. Or any other committer can do it too.
Comment 16 Yuri Dario 2015-09-10 14:01:48 UTC
Ok, I can merge my patches. What about patches for other tickets? can I merge them freely or I need some kind of authorization?
Comment 17 Andrea Pescetti 2015-09-10 14:22:38 UTC
Everything that is relevant for 4.1.2 should be flagged with "Flags: 4.1.2 release blocker : ?". This triggers a mail to the dev list.

Once, after possible discussion, the "?" is turned into a "+" (which means: the release blocker is approved) then anyone can merge the fix to AOO410.

For 4.1.2 we tend to only allow important bugfixes or, to be more precise, bugfixes that do not carry a significant risk of side-effects.
Comment 18 Yuri Dario 2015-09-10 15:26:06 UTC
All fixes I did in the trunk has been added on my personal branch for AOO 4.1.x, and the current OS/2 build is done using all existing trunk patches.

Also patches are OS/2 only, they don't affect other code.

So instead of merging to AOO branch, I can simply update my local branch as before. Just to avoid more discussions and slow down things.
Comment 19 Kay 2015-09-10 21:53:30 UTC
(In reply to Andrea Pescetti from comment #17)
> Everything that is relevant for 4.1.2 should be flagged with "Flags: 4.1.2
> release blocker : ?". This triggers a mail to the dev list.
> 
> Once, after possible discussion, the "?" is turned into a "+" (which means:
> the release blocker is approved) then anyone can merge the fix to AOO410.
> 
> For 4.1.2 we tend to only allow important bugfixes or, to be more precise,
> bugfixes that do not carry a significant risk of side-effects.

Wouldn't it be less confusing in the long run to merge this OS/2 patch into the AOO410 trunk unless Yuri does all his builds from trunk normally. I was thinking if he (or anyone) releases an AOO 4.1.2 port it should be built from the correct branch and using the tarball that will be supplied.
Comment 20 Andrea Pescetti 2015-09-10 22:09:27 UTC
Yuri, I didn't want to put any process here. Does trunk build under OS/2? Nice. Do you have a well-defined set of patches that could be applied to make the AOO410 branch build under OS/2? Perfect. Just re-apply those patches then, and we have a 4.1.2 source tarball that builds fine under OS/2.

I don't see this as slowing down things. If you have commits on trunk that help the OS/2 build just merge them to AOO410. Even if they go beyond the three patches listed in this issue. But with the condition that all of them are, as you say, specific to OS/2 builds.

In other words, should we move the action from here to the "parent issue" https://bz.apache.org/ooo/show_bug.cgi?id=118923 ?
Comment 21 Yuri Dario 2015-09-11 13:44:45 UTC
Yes, the parent issue is the main tracker for os2 activity. Also it references some closed tickets whose changes are only in trunk.

Should I add a '?' release blocker to all relevant tickets? (some older tickets are already merged to branch).
Comment 22 Andrea Pescetti 2015-09-11 14:10:08 UTC
No need for special process, just feel free to go ahead and commit the relevant patches from the right issues. It will be easy (and lightweight) if you set "Target: 4.1.2" in those issues, but consider it a "nice to have".

I've sent the following clarification to the dev list:
   ---
As clarified with Yuri Dario (see issue): the commits in this specific issue are part of a bigger set of commits that are already in trunk and that Yuri can port to the AOO410 branch so that OpenOffice 4.1.2 will build on OS/2.

All patches are OS/2 specific. The "blocker" flag is to be considered automatically extended to related issues and patches. Yuri Dario will personally pick the right ones and commit them to the AOO410 branch.
   ---