Skip to contentSkip to conversation

package/libical: bump version to v1.0.1

PATCH New Bernd Kuhls 2 participants 2 messages Apr 26, 2015 – Apr 27, 2015

lore
Share & tools
lore
#0 lore raw

PATCH v3 1/1 package/libical: bump version to v1.0.1

Project moved to github according to http://sourceforge.net/projects/freeassociation
"As of 2014-06-14, this project may now be found at http://github.com/libical."

From install.txt:

Libical v1.0 and above uses the CMake buildsystem.
The autoconf build system has been removed and is no longer supported.

---
v3: add note about cmake switch in commit message (Baruch)
v2: adjust comment in Config.in (Baruch)

 package/libical/0001-no-tests.patch |   19 +++++++++++++++++++
 package/libical/Config.in           |    7 ++++---
 package/libical/libical.hash        |    2 +-
 package/libical/libical.mk          |   10 +++++++---
 4 files changed, 31 insertions(+), 7 deletions(-)
 create mode 100644 package/libical/0001-no-tests.patch
package/libical/0001-no-tests.patch new file +19 -0
new file mode 100644
index 0000000..324c04f
@@ -0,0 +1,19 @@
+Disable tests to avoid MMU dependency due to fork.
+
+Fixes
+CMakeFiles/regression.dir/regression.c.o: In function `_test_file_locks':
+/home/br/br4/output/build/libical-v1.0.1/src/test/regression.c:(.text+0x96d8): undefined reference to `_fork'
+
+Signed-off-by: Bernd Kuhls <[email protected]>
+
+diff -uNr libical-1.0.1.org/src/CMakeLists.txt libical-1.0.1/src/CMakeLists.txt
+--- libical-1.0.1.org/src/CMakeLists.txt	2014-10-09 17:07:05.000000000 +0200
++++ libical-1.0.1/src/CMakeLists.txt	2015-04-25 17:41:22.657345548 +0200
+@@ -1,7 +1,6 @@
+ add_subdirectory(libical)
+ add_subdirectory(libicalss)
+ add_subdirectory(libicalvcal)
+-add_subdirectory(test)
+ 
+ if(MSVC)
+   install(FILES
package/libical/Config.in +4 -3
index 9bb63c3..1616d34 100644
@@ -1,11 +1,12 @@
 config BR2_PACKAGE_LIBICAL
 	bool "libical"
+	depends on !BR2_STATIC_LIBS
 	depends on BR2_USE_WCHAR
 	help
 	  libical is an Open Source (MPL/LGPL) implementation of the IETF's
 	  iCalendar Calendaring and Scheduling protocols.
 
-	  http://www.citadel.org/doku.php/documentation:featured_projects:libical
+	  http://libical.github.io/libical
 
-comment "libical needs a toolchain w/ wchar"
-	depends on !BR2_USE_WCHAR
+comment "libical needs a toolchain w/ dynamic library, wchar"
+	depends on !BR2_USE_WCHAR || BR2_STATIC_LIBS
package/libical/libical.hash +1 -1
index 2382ad7..ab5c2dd 100644
@@ -1,2 +1,2 @@
 # Locally computed:
-sha256  2ae78b0757f0dd13431acf42a9a8d038339fd4767fd5134e650bf60ee0b4dff0  libical-0.48.tar.gz
+sha256	7d5f613454ec6c7d1bcfb441c919215be53292aa15cd1cb14249d1413d6c610c	libical-v1.0.1.tar.gz
package/libical/libical.mk +7 -3
index 4025627..b3702ae 100644
@@ -4,10 +4,14 @@
 #
 ################################################################################
 
-LIBICAL_VERSION = 0.48
-LIBICAL_SITE = http://downloads.sourceforge.net/project/freeassociation/libical/libical-$(LIBICAL_VERSION)
+LIBICAL_VERSION = v1.0.1
+LIBICAL_SITE = $(call github,libical,libical,$(LIBICAL_VERSION))
 LIBICAL_INSTALL_STAGING = YES
 LIBICAL_LICENSE = MPLv1.0 or LGPLv2.1
 LIBICAL_LICENSE_FILES = LICENSE
 
-$(eval $(autotools-package))
+# building without this option is broken, it is used by
+# Gentoo/alpinelinux as well
+LIBICAL_CONF_OPTS = -DSHARED_ONLY=true
+
+$(eval $(cmake-package))
Signature · 1 line
1.7.10.4
#1 lore raw

PATCH v3 1/1 package/libical: bump version to v1.0.1

Dear Bernd Kuhls,

package/libical/libical.hash line 2

Quoted 4 lines · Bernd Kuhls
package/libical/libical.hash +1 -1
@@ -1,2 +1,2 @@
 # Locally computed:
-sha256  2ae78b0757f0dd13431acf42a9a8d038339fd4767fd5134e650bf60ee0b4dff0  libical-0.48.tar.gz
+sha256	7d5f613454ec6c7d1bcfb441c919215be53292aa15cd1cb14249d1413d6c610c	libical-v1.0.1.tar.gz

A hash with a $(call github,...) <pkg>_SITE seems wrong.

Also, from the manual:

"""
If the package you wish to add does have a release section on GitHub, the maintainer may have uploaded a release tarball, or the release may just point to the automatically generated tarball from the git tag. If there is a release tarball uploaded by the maintainer, we prefer to use that since it may be slightly different (e.g. it contains a configure script so we don?t need to do AUTORECONF).

You can see on the release page if it?s an uploaded tarball or a git tag:

* If there is a green download button, like mongrel2, then it was uploaded by the maintainer and you should use the link of that button to specify FOO_SITE, and not use the github helper.

* If there is grey download button, like xbmc, then it?s an automatically generated tarball and you should use the github helper function.
"""

libical seems to be inthe first case, i.e we should be able to have a hash file, but we shouldn't use the github function.

Thomas

Signature · 3 lines
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com