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 2719 - libkcal tests fails
Summary: libkcal tests fails
Status: RESOLVED FIXED
Alias: None
Product: TDE
Classification: Unclassified
Component: tdepim (show other bugs)
Version: R14.0.x [Trinity]
Hardware: All All
: P5 normal
Assignee: Michele Calgaro
URL:
Depends on:
Blocks: R14.0.7
  Show dependency tree / graph
 
Reported: 2016-10-15 19:42 CDT by Slávek Banko
Modified: 2019-04-01 17:01 CDT (History)
4 users (show)

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

Attachments
cmake: add libkcal tests during building (7.93 KB, patch)
2016-10-15 19:42 CDT, Slávek Banko 		Details | Diff
fix the building of libkcal/tests (405 bytes, patch)
2017-01-14 18:35 CST, deloptes 		Details | Diff
Show Obsolete (2) 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 Slávek Banko 2016-10-15 19:42:23 CDT 
Created attachment 2733 [details]
cmake: add libkcal tests during building

This bug report continues to bug 2660, in which were also dealt libkcal tests during build. The attached patch adds tests to CMake build, but tests currently fail. Therefore, the patch has not yet been pushed to git.
Comment 1 deloptes 2017-01-14 18:35:52 CST 
Created attachment 2748 [details]
fix the building of libkcal/tests

apply this patch in libkcal dir to update CMakeLists.txt
Comment 2 Slávek Banko 2017-01-14 18:41:15 CST 
(In reply to deloptes from comment #1)
> Created attachment 2748 [details]
> fix the building of libkcal/tests
> 
> apply this patch in libkcal dir to update CMakeLists.txt

You're trying, that such a patch will solve the problem with the tests? Because I did not notice a problem with build, but with the test results.
Comment 3 deloptes 2017-01-15 02:12:28 CST 
Thanks for pointing this out
I set up a clean build environment. I guess those headers were picked up from somewhere else before. Let me check if tests run as well - I did not pay attention.
Comment 4 deloptes 2017-01-15 18:01:55 CST 
The code did not build in clean env, so the patch is needed to be able to build the project in clean env anyway. Another topic is the execution of the tests.
So you are correct. The patch needs more work. Now that I have a build env I hope I can improve it. I put it on my todo and I hope I'll have some time next to do it.

regards
Comment 5 deloptes 2019-03-13 03:25:54 CDT 
Most recent update on this is in Gitea https://mirror.git.trinitydesktop.org/gitea/TDE/tdepim/pulls/17
Comment 6 Michele Calgaro 2019-03-24 07:44:43 CDT 
Reworked PR here:
https://mirror.git.trinitydesktop.org/gitea/TDE/tdepim/pulls/18