On Sat, Aug 03, 2019 at 05:13:55PM +0200, Arnold Schulz wrote:
> This solves:
> - Fix build failure due to missing telephony.h in kernel 4.19 sources
> - OpenWrt specific patch+file removed; this stuff is now in upstream
>
> Signed-off-by: Arnold Schulz <[email protected]>
> ---
>  net/asterisk-16.x-chan-lantiq/Makefile        | 16 +++----
>  .../files/default.exports                     |  8 ----
>  ...-ast_free-instead-of-malloc-and-free.patch | 43 -------------------
>  3 files changed, 5 insertions(+), 62 deletions(-)
>  delete mode 100644 net/asterisk-16.x-chan-lantiq/files/default.exports
>  delete mode 100644 
> net/asterisk-16.x-chan-lantiq/patches/01-Use-ast_malloc-and-ast_free-instead-of-malloc-and-free.patch
>
> diff --git a/net/asterisk-16.x-chan-lantiq/Makefile 
> b/net/asterisk-16.x-chan-lantiq/Makefile
> index 5884673..8210c07 100644
> --- a/net/asterisk-16.x-chan-lantiq/Makefile
> +++ b/net/asterisk-16.x-chan-lantiq/Makefile
> @@ -1,5 +1,5 @@
>  #
> -# Copyright (C) 2018 OpenWrt.org
> +# Copyright (C) 2018-2019 OpenWrt.org

Are you a member of OpenWrt? Only members should should change OpenWrt
copyright message.

>  #
>  # This is free software, licensed under the GNU General Public License v2.
>  # See /LICENSE for more information.
> @@ -8,14 +8,14 @@
>  include $(TOPDIR)/rules.mk
>
>  PKG_NAME:=asterisk16-chan-lantiq
> -PKG_VERSION:=20180215
> -PKG_RELEASE:=2
> +PKG_VERSION:=20190803
> +PKG_RELEASE:=1
>
>  PKG_SOURCE:=$(PKG_NAME)-$(PKG_VERSION).tar.xz
>  PKG_SOURCE_URL:=https://github.com/kochstefan/asterisk_channel_lantiq.git
>  PKG_SOURCE_SUBDIR:=$(PKG_NAME)-$(PKG_VERSION)
> -PKG_SOURCE_VERSION:=c9d68dd06fcd46ac7985df45f8c2b8833e658f8e
> -PKG_MIRROR_HASH:=8666c18b24adf9da3ddf12306fcf0a8b4f56860c256b172bd0ba5c2a7a3ab25e
> +PKG_SOURCE_VERSION:=1d940b38cde0348dfe129d2b764e6faee440c45b
> +PKG_MIRROR_HASH:=ff838ff2a4c5353fadd73806e1513f59f224914582b6ba004165712268bc94e5
>  PKG_SOURCE_PROTO:=git
>
>  PKG_LICENSE:=GPL-2.0
> @@ -43,12 +43,6 @@ define Package/$(PKG_NAME)/conffiles
>  /etc/asterisk/lantiq.conf
>  endef
>
> -define Build/Prepare
> -     $(call Build/Prepare/Default)
> -     $(INSTALL_DATA) ./files/default.exports \
> -             $(PKG_BUILD_DIR)/src/channels/chan_lantiq.exports
> -endef
> -

Why remove this? I put this in to recreate what happens during an
in-tree build (adding chan-lantig files into Asterisk source tree and
compiling it with Asterisk).

>  define Build/Compile
>       cd $(PKG_BUILD_DIR)/src/channels && \
>       $(TARGET_CC) -o chan_lantiq.o -c chan_lantiq.c -MD -MT chan_lantiq.o \
> diff --git a/net/asterisk-16.x-chan-lantiq/files/default.exports 
> b/net/asterisk-16.x-chan-lantiq/files/default.exports
> deleted file mode 100644
> index 6d9344d..0000000
> --- a/net/asterisk-16.x-chan-lantiq/files/default.exports
> +++ /dev/null
> @@ -1,8 +0,0 @@
> -{
> -     global:
> -             /* See main/asterisk.exports.in for an explanation why this is
> -              * needed. */
> -             _IO_stdin_used;
> -     local:
> -             *;
> -};
> diff --git 
> a/net/asterisk-16.x-chan-lantiq/patches/01-Use-ast_malloc-and-ast_free-instead-of-malloc-and-free.patch
>  
> b/net/asterisk-16.x-chan-lantiq/patches/01-Use-ast_malloc-and-ast_free-instead-of-malloc-and-free.patch
> deleted file mode 100644
> index f913b18..0000000
> --- 
> a/net/asterisk-16.x-chan-lantiq/patches/01-Use-ast_malloc-and-ast_free-instead-of-malloc-and-free.patch
> +++ /dev/null
> @@ -1,43 +0,0 @@
> -commit 30f9a094c1c60e0d68e4ea189f48ecb47aebb485
> -Author: arny <[email protected]>
> -Date:   Thu May 2 20:07:28 2019 +0200
> -
> -    Use ast_malloc and ast_free instead of malloc and free
> -    in order to get rid of build errors with asterisk16 in OpenWrt
> -
> -    Signed-off-by: arny <[email protected]>
> -
> -diff --git a/src/channels/chan_lantiq.c b/src/channels/chan_lantiq.c
> -index a8fc869..90002ab 100644
> ---- a/src/channels/chan_lantiq.c
> -+++ b/src/channels/chan_lantiq.c
> -@@ -563,9 +563,9 @@ lantiq_dev_binary_buffer_create(const char *path, 
> uint8_t **ppBuf, uint32_t *pBu
> -             goto on_exit;
> -     }
> -
> --    *ppBuf = malloc(file_stat.st_size);
> -+    *ppBuf = ast_malloc(file_stat.st_size);
> -     if (*ppBuf == NULL) {
> --            ast_log(LOG_ERROR, "binary file %s memory allocation failed\n", 
> path);
> -+            // Message already logged by ast_malloc
> -             goto on_exit;
> -     }
> -
> -@@ -583,7 +583,7 @@ on_exit:
> -             fclose(fd);
> -
> -     if (*ppBuf != NULL && status)
> --            free(*ppBuf);
> -+            ast_free(*ppBuf);
> -
> -     return status;
> - }
> -@@ -609,7 +609,7 @@ static int32_t lantiq_dev_firmware_download(int32_t fd, 
> const char *path)
> -     }
> -
> -     if (firmware != NULL)
> --            free(firmware);
> -+            ast_free(firmware);
> -
> -     return 0;
> - }
> --
> 2.20.1
>
>
> _______________________________________________
> openwrt-devel mailing list
> [email protected]
> https://lists.openwrt.org/mailman/listinfo/openwrt-devel

_______________________________________________
openwrt-devel mailing list
[email protected]
https://lists.openwrt.org/mailman/listinfo/openwrt-devel

Reply via email to