HAVE_GNOKII_H is used but never checked with CMake.
Addiitonnaly, "version 0.6.13 or later" is suggested, but currently only such version are detected as described above.
0.6.12 or older version must be supported?
If so, ConfigureChecks.cmake should be changed to detect even if missing gnokii.pc, and declear HAVE_GNOKII_H for newer version.
If not, using parts such as gnokii_xxport.cpp should be changed as using codes for 0.6.13 or later unconditionally, and remove codes for older releases.
Steps to reproduce
configure and build with BUILD_KADDRESSBOOK=ON
rise warnings missing and older gnokii even if using newser version with development header and libraries
Screenshots
<!--
This is a comment.
Please fill in the required fields below.
The comments provide instructions on how to do so.
Note: You do not need to remove comments.
-->
## Basic information
- TDE version: <!-- such as R14.0.9 - see tde-config -v -->
- Distribution: <!-- such as Debian Stretch - see lsb_release -sd -->
- Hardware: <!-- amd64 / i386 / armhf / ... -->
<!--
Use SL/* labels to set the severity level.
Please do not set a milestone.
-->
## Description
Currently, gnokii will be checked with ` pkg_search_module`
https://mirror.git.trinitydesktop.org/gitea/TDE/tdepim/src/commit/4ad5e8c72cf9c6956a326c6011ef3cfad7bb78c3/ConfigureChecks.cmake#L58-L63
It will use `pkg-config`, and `gnokii.pc` is provided by gnokii version 0.6.13 or later.
In tdepim/kaddressbook/xxport/gnokii_xxport.cpp
https://mirror.git.trinitydesktop.org/gitea/TDE/tdepim/src/commit/4ad5e8c72cf9c6956a326c6011ef3cfad7bb78c3/kaddressbook/xxport/gnokii_xxport.cpp#L35-L44
`HAVE_GNOKII_H` is used but never checked with CMake.
Addiitonnaly, "version 0.6.13 or later" is suggested, but currently only such version are detected as described above.
0.6.12 or older version must be supported?
If so, ConfigureChecks.cmake should be changed to detect even if missing gnokii.pc, and declear HAVE_GNOKII_H for newer version.
If not, using parts such as `gnokii_xxport.cpp` should be changed as using codes for 0.6.13 or later unconditionally, and remove codes for older releases.
## Steps to reproduce
1. configure and build with BUILD_KADDRESSBOOK=ON
2. rise warnings missing and older gnokii even if using newser version with development header and libraries
## Screenshots
<!-- If it seems useful, please provide provide one or more screenshots. -->
Does someone use gnokii? Does it have any application in todays world?
I don't use it but I see the package still available in debian repos... I guess we can assume someone is still looking after it
> Does someone use gnokii? Does it have any application in todays world?
I don't use it but I see the package still available in debian repos... I guess we can assume someone is still looking after it
Does someone use gnokii? Does it have any application in todays world?
I don't use it but I see the package still available in debian repos... I guess we can assume someone is still looking after it
Because there is a build option WITH_GNOKII, then it is in place to test for gnokii as needed to be properly functional.
> > Does someone use gnokii? Does it have any application in todays world?
>
> I don't use it but I see the package still available in debian repos... I guess we can assume someone is still looking after it
Because there is a build option `WITH_GNOKII`, then it is in place to test for gnokii as needed to be properly functional.
It will use pkg-config, and gnokii.pc is provided by gnokii version 0.6.13 or later.
I is wrong, xgnokii.pc is provided by gnokii version 0.6.13 or later,
but gnokii.pc is version 0.5.0 or later.
0.6.12 or older version must be supported?
0.4.x or older version must be supported?
Yes, no need to support over 19 years old releases.
> It will use `pkg-config`, and `gnokii.pc` is provided by gnokii version 0.6.13 or later.
I is wrong, `xgnokii.pc` is provided by gnokii version 0.6.13 or later,
but `gnokii.pc` is version 0.5.0 or later.
> 0.6.12 or older version must be supported?
0.4.x or older version must be supported?
Yes, no need to support over 19 years old releases.
Basic information
Description
Currently, gnokii will be checked with
pkg_search_module
4ad5e8c72c/ConfigureChecks.cmake (L58-L63)
It will use
pkg-config
, andgnokii.pc
is provided by gnokii version 0.6.13 or later.In tdepim/kaddressbook/xxport/gnokii_xxport.cpp
4ad5e8c72c/kaddressbook/xxport/gnokii_xxport.cpp (L35-L44)
HAVE_GNOKII_H
is used but never checked with CMake.Addiitonnaly, "version 0.6.13 or later" is suggested, but currently only such version are detected as described above.
0.6.12 or older version must be supported?
If so, ConfigureChecks.cmake should be changed to detect even if missing gnokii.pc, and declear HAVE_GNOKII_H for newer version.
If not, using parts such as
gnokii_xxport.cpp
should be changed as using codes for 0.6.13 or later unconditionally, and remove codes for older releases.Steps to reproduce
Screenshots
Does someone use gnokii? Does it have any application in todays world?
Just asking.
I don't use it but I see the package still available in debian repos... I guess we can assume someone is still looking after it
Because there is a build option
WITH_GNOKII
, then it is in place to test for gnokii as needed to be properly functional.This is precisly why I was asking about this, but it looks like it might be still possible to send/read SMS with it.
Fixed by PR #57.
I is wrong,
xgnokii.pc
is provided by gnokii version 0.6.13 or later,but
gnokii.pc
is version 0.5.0 or later.0.4.x or older version must be supported?
Yes, no need to support over 19 years old releases.