Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

deprecate autotools #10797

Merged
merged 1 commit into from Oct 30, 2016
Merged

deprecate autotools #10797

merged 1 commit into from Oct 30, 2016

Conversation

wsnipex
Copy link
Member

@wsnipex wsnipex commented Oct 28, 2016

Give a fair warning that autotools will die soon

@MartijnKaijser MartijnKaijser added this to the Krypton 17.0-beta6 milestone Oct 28, 2016
@@ -2408,3 +2408,8 @@ AC_OUTPUT

final_message="$final_message\n prefix:\t$prefix\n$dashes"
echo -e "$final_message\n"
echo "#------------------------------------ WARNING ----------------------------------------#"
echo "The autotools based build system is deprecated and will be removed in the next version!"

This comment was marked as spam.

This comment was marked as spam.

This comment was marked as spam.

@wsnipex wsnipex merged commit bc895f3 into xbmc:master Oct 30, 2016
woodsts pushed a commit to woodsts/buildroot that referenced this pull request Apr 29, 2017
Removed unneeded patches
- 0001-Fixup-include-path.patch (not needed after CMake switch)
- 0005-native-TexturePacker-fix-compilation-with-gcc-4.6.patch
  (applied upstream)
- 0006-ffmpeg30.patch (was backported from 17.0-Krypton to 16.0-Jarvis)
- 0007-exif-Fix-for-out-of-memory-errors-with-large-numbers.patch
  (was backported from 17.0-Krypton to 16.0-Jarvis)
- 0008-Fix-nullpadding-issue-when-reading-certain-id3v1-tag.patch
  (was backported from 17.0-Krypton to 16.0-Jarvis)
- 0009-lib-cximage-6.0-fix-compilation-with-gcc6.patch
  (cximage was removed in bump from 16.x to 17.0)
- 0010-curl-support-version-7.5.0-and-upwards.patch
  (applied upstream)
- 0011-xbmc_pvr_types.h-Fix-compilation-with-gcc6.patch
  (applied upstream)
- 0012-Fix_includes_in_amcodec.patch
  (was backported from 17.0-Krypton to 16.0-Jarvis)

Rebased patches
- 0004-kodi-config.cmake-use-CMAKE_FIND_ROOT_PATH-to-fix-cr.patch
  also renamed to 0001-...

Removed dependencies not needed anymore:
- boost
  xbmc/xbmc@41ae93f
- giflib
  xbmc/xbmc@d44338b
- jasper/tiff
  xbmc/xbmc@00724eb
- jpeg
  xbmc/xbmc@7d5bdfb
- libdcadec
  xbmc/xbmc@378eb26
- libglew
  xbmc/xbmc@03ff0d5
- libgcrypt
  was already an optional dependency in Kodi 16, not part of the CMake
  buildsystem anymore
- libmpeg2
  xbmc/xbmc@d22c829
- libogg/libvorbis
  xbmc/xbmc@4c60969
- libpng
  xbmc/xbmc@be6b50c
- librtmp, the new rtmp inputstream addon will be added later
  xbmc/xbmc@d04f43a
- libsquish
  xbmc/xbmc@ed03f82
- xlib_libXmu
- xlib_libXt

Switched to CMake, autoconf was deprecated:
xbmc/xbmc#10797

The dependency for egl/gles on arm, formerly enforced by the automake
build system, was not ported to CMake.

Bumped BR2_TOOLCHAIN_GCC_AT_LEAST to 4.8 to fix build errors with
gcc-4.7 found while testing
http://autobuild.buildroot.net/toolchains/configs/sourcery-x86.config
For details please read
http://lists.busybox.net/pipermail/buildroot/2017-April/190195.html

Added hard-dependency for libegl, needed after
xbmc/xbmc@0ac305f

Libva support depends on X11
https://github.com/xbmc/xbmc/blob/Krypton/project/cmake/modules/FindVAAPI.cmake#L42
and OpenGL/EGL
https://github.com/xbmc/xbmc/blob/Krypton/xbmc/cores/VideoPlayer/DVDCodecs/Video/VAAPI.h#L23

Libvdpau support depends on X11
https://github.com/xbmc/xbmc/blob/Krypton/project/cmake/modules/FindVDPAU.cmake#L21
and OpenGL/EGL
https://github.com/xbmc/xbmc/blob/Krypton/xbmc/cores/VideoPlayer/DVDCodecs/Video/VDPAU.h#L43

Updated clean-up hook and added host-xmlstarlet as dependency to
manipulate the list of default system addons in addon-manifest.xml.

Added dependency to BR2_ENABLE_LOCALE, needs iconv_open:
https://github.com/xbmc/xbmc/blob/Krypton/xbmc/utils/CharsetConverter.cpp#L200

Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de>
[Thomas: minor tweaks.]
Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants