Skip to content

Commit

Permalink
Merge pull request #30 from AlwinEsch/Leia-change
Browse files Browse the repository at this point in the history
[Leia] some cleanups
  • Loading branch information
AlwinEsch committed Feb 6, 2020
2 parents ccff3e0 + a22a777 commit 8e6c74e
Show file tree
Hide file tree
Showing 9 changed files with 47 additions and 9 deletions.
2 changes: 0 additions & 2 deletions .travis.yml
Expand Up @@ -17,8 +17,6 @@ matrix:
dist: xenial
sudo: required
compiler: clang
- os: osx
osx_image: xcode9
- os: osx
osx_image: xcode9.4

Expand Down
1 change: 1 addition & 0 deletions README.md
Expand Up @@ -4,6 +4,7 @@ This is a [Kodi] (http://kodi.tv) Vorbis audio encoder add-on.

#### CI Testing
[![Build Status](https://travis-ci.org/xbmc/audioencoder.vorbis.svg?branch=master)](https://travis-ci.org/xbmc/audioencoder.vorbis)
[![Build Status](https://dev.azure.com/teamkodi/binary-addons/_apis/build/status/xbmc.audioencoder.vorbis?branchName=Leia)](https://dev.azure.com/teamkodi/binary-addons/_build/latest?definitionId=23&branchName=Leia)
[![Coverity Scan Build Status](https://scan.coverity.com/projects/5120/badge.svg)](https://scan.coverity.com/projects/5120)

## Build instructions
Expand Down
7 changes: 6 additions & 1 deletion audioencoder.vorbis/addon.xml.in
@@ -1,7 +1,7 @@
<?xml version="1.0" encoding="UTF-8"?>
<addon
id="audioencoder.vorbis"
version="2.0.3"
version="2.0.4"
name="Vorbis Audio Encoder"
provider-name="spiff">
<requires>@ADDON_DEPENDS@</requires>
Expand All @@ -13,5 +13,10 @@
<summary lang="en">Vorbis audio compression</summary>
<description lang="en">Vorbis is a general purpose perceptual audio CODEC intended to allow maximum encoder flexibility, thus allowing it to scale competitively over an exceptionally wide range of bitrates.</description>
<platform>@PLATFORM@</platform>
<license>GPL-2.0</license>
<source>https://github.com/xbmc/audioencoder.vorbis</source>
<assets>
<icon>icon.png</icon>
</assets>
</extension>
</addon>
2 changes: 1 addition & 1 deletion debian/compat
@@ -1 +1 @@
8
9
6 changes: 3 additions & 3 deletions debian/control
@@ -1,8 +1,8 @@
Source: kodi-audioencoder-vorbis
Priority: extra
Maintainer: Arne Morten Kvarving <spiff@xbmc.org>
Build-Depends: debhelper (>= 8.0.0), cmake, kodi-addon-dev, make, libtool, libvorbis-dev
Standards-Version: 3.9.6
Maintainer: Nobody <nobody@kodi.tv>
Build-Depends: debhelper (>= 9.0.0), cmake, kodi-addon-dev, make, libtool, libvorbis-dev
Standards-Version: 4.1.2
Section: libs
Homepage: http://kodi.org

Expand Down
2 changes: 2 additions & 0 deletions debian/copyright
Expand Up @@ -3,6 +3,7 @@ Upstream-Name: audioencoder.vorbis

Files: *
Copyright: Arne Morten Kvarving <arne.morten.kvarving@sintef.no>
2013-2020 Team Kodi
License: GPL-2+
This package is free software; you can redistribute it and/or modify
it under the terms of the GNU General Public License as published by
Expand All @@ -24,6 +25,7 @@ License: GPL-2+
Files: debian/*
Copyright: 2013 Arne Morten Kvarving <arne.morten.kvarving@sintef.no>
2013 wsnipex <wsnipex@a1.net>
2013-2020 Team Kodi
License: GPL-2+
This package is free software; you can redistribute it and/or modify
it under the terms of the GNU General Public License as published by
Expand Down
2 changes: 1 addition & 1 deletion debian/source/format
@@ -1 +1 @@
3.0 (native)
3.0 (quilt)
32 changes: 32 additions & 0 deletions depends/common/vorbis/0001-fix-windows-arm-build.patch
@@ -0,0 +1,32 @@
--- a/lib/CMakeLists.txt
+++ b/lib/CMakeLists.txt
@@ -65,6 +65,18 @@ if(MSVC)
list(APPEND VORBIS_SOURCES ../win32/vorbis.def)
list(APPEND VORBISENC_SOURCES ../win32/vorbisenc.def)
list(APPEND VORBISFILE_SOURCES ../win32/vorbisfile.def)
+
+ include(CheckSymbolExists)
+ check_symbol_exists(_X86_ "Windows.h" _X86_)
+ check_symbol_exists(_AMD64_ "Windows.h" _X64_)
+ check_symbol_exists(_ARM_ "Windows.h" _ARM_)
+ check_symbol_exists(_ARM64_ "Windows.h" _ARM64_)
+
+ if(_ARM64_)
+ add_definitions(-D_M_ARM -D_M_ARM64)
+ elseif(_ARM_)
+ add_definitions(-D_M_ARM)
+ endif()
endif()

include_directories(../include)
--- a/lib/os.h
+++ b/lib/os.h
@@ -147,7 +147,7 @@ static __inline void vorbis_fpu_restore(vorbis_fpu_control fpu){

/* Optimized code path for x86_64 builds. Uses SSE2 intrinsics. This can be
done safely because all x86_64 CPUs supports SSE2. */
-#if (defined(_MSC_VER) && defined(_WIN64)) || (defined(__GNUC__) && defined (__x86_64__))
+#if (defined(_MSC_VER) && defined(_WIN64) && !defined(_M_ARM)) || (defined(__GNUC__) && defined (__x86_64__))
# define VORBIS_FPU_CONTROL

typedef ogg_int16_t vorbis_fpu_control;
2 changes: 1 addition & 1 deletion src/EncoderVorbis.cpp
@@ -1,5 +1,5 @@
/*
* Copyright (C) 2005-2019 Team Kodi
* Copyright (C) 2005-2020 Team Kodi
* http://kodi.tv
*
* This Program is free software; you can redistribute it and/or modify
Expand Down

0 comments on commit 8e6c74e

Please sign in to comment.