By default, Bugzilla does not search the list of RESOLVED bugs.
You can force it to do so by putting the upper-case word ALL in front of your search query, e.g.: ALL tdelibs
We recommend searching for bugs this way, as you may discover that your bug has already been resolved and fixed in a later release.
Bug 627 - Build issue: Kspell Sym Link Incorrect
Summary: Build issue: Kspell Sym Link Incorrect
Status: RESOLVED FIXED
Alias: None
Product: TDE
Classification: Unclassified
Component: tdelibs (show other bugs)
Version: R14.0.0 [Trinity]
Hardware: Other All
: P1 normal
Assignee: Timothy Pearson
URL:
Depends on:
Blocks:
 
Reported: 2011-11-16 00:44 CST by Darrell
Modified: 2013-02-16 11:51 CST (History)
4 users (show)

See Also:
Compiler Version:
TDE Version String:
Application Version:
Application Name:

Attachments
kdelibs: fix 'tde_create_handbook' macro if TDE_HTML_DIR is not set (498 bytes, patch)
2013-02-16 08:13 CST, Francois Andriot 		Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Darrell 2011-11-16 00:44:22 CST 
In the $PREFIX/share/doc/kde/HTML/en/kspell/common directory, the sym link to
the common HTML files are incorrect.

Refer to Bug 347 for the same problem.

Paper cut candidate.
Comment 1 Francois Andriot 2013-02-16 08:05:07 CST 
In source file 'doc/kspell/CMakeLists.txt' , look at line:
  tde_create_handbook( SRCDIR ${CMAKE_SOURCE_DIR}/kdoctools DESTINATION kspell )
We use the 'tde_create_handbook' macro.

Now look at macro 'tde_create_handbook' in 'cmake/TDEMacros.cmake' : It is using a variable 'TDE_HTML_DIR' to create the symlink.
Since the resulting link is '/en/common', it looks like the variable TDE_HTML_DIR is empty.

This variable is set in 'cmake/FindTDE.cmake', using the 'kde-config' command.

We have a build issue here, because the 'kde-config' binary is installed by the kdelibs package ... So in order to build kdelibs correctly, we need that kdelibs is already installed !
Comment 2 Francois Andriot 2013-02-16 08:13:45 CST 
Created attachment 1108 [details]
kdelibs: fix 'tde_create_handbook' macro if TDE_HTML_DIR is not set

This patch use variable HTML_INSTALL_DIR in tde_create_handbook macro, is TDE_HTML_DIR is not set. Now kspell 'common' link is correct.
Comment 3 Slávek Banko 2013-02-16 11:51:15 CST 
François, good job, thank you.
Fixed in GIT hash b928573e and also in v3.5.13-sru branch.