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
Continuation of @jschuelller's 5.12.4 PR. #120
Changes from 8 commits
aaa15ed
727fee5
6fb0706
2ff78f8
50afc64
4e2f5f3
fd230d9
a3a7383
009f088
6cb95a5
e85a642
b46ed83
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,2 +1,4 @@ | ||
CONDA_BUILD_SYSROOT: # [osx] | ||
- /opt/MacOSX10.12.sdk # [osx] | ||
MACOSX_DEPLOYMENT_TARGET: # [osx] | ||
- '10.12' # [osx] |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -3,29 +3,45 @@ | |
# error: unknown type name 'SecKeyAlgorithm' | ||
# .. these 10.11 support patches should be removed anyway as they could break some things. | ||
# .. such as pop-up Window mouse-events. | ||
{% set version = "5.12.1" %} | ||
{% set version = "5.12.4" %} | ||
{% set build_number = "0" %} | ||
{% set dev_nature = os.environ.get('CONDA_BUILD_DEBUG_PACKAGE', '') %} | ||
{% if dev_nature != '' %} | ||
{% set dev_nature = "_dev" %} | ||
{% set patches = "patches/git" %} | ||
{% else %} | ||
{% set patches = "patches" %} | ||
{% endif %} | ||
|
||
package: | ||
name: qt | ||
version: {{ version }} | ||
|
||
source: | ||
- url: http://download.qt.io/official_releases/qt/5.12/{{ version }}/single/qt-everywhere-src-{{ version }}.tar.xz # [not win] | ||
md5: 6a37466c8c40e87d4a19c3f286ec2542 # [not win] | ||
- url: http://download.qt.io/official_releases/qt/5.12/{{ version }}/single/qt-everywhere-src-{{ version }}.zip # [win] | ||
md5: 4649d4e51ca836fbde08565582353140 # [win] | ||
{% if dev_nature == "_dev" %} | ||
- git_url: git://code.qt.io/qt/qt5.git | ||
git_tag: {{ version }} | ||
{% else %} | ||
- url: http://download.qt.io/official_releases/qt/5.12/{{ version }}/single/qt-everywhere-src-{{ version }}.tar.xz | ||
md5: dda95b0239d13c5276834177af3a8588 | ||
{% endif %} | ||
|
||
patches: | ||
|
||
# qtbase | ||
- patches/0010-osx-xctest-check.patch | ||
- patches/0011-osx-allow-any-xcrun-in-PATH.patch | ||
- patches/0015-qtbase-use-better-clang-optimize-size.patch | ||
- patches/0016-qtbase-use-better-gcc-optimize-size.patch | ||
- patches/0017-qtbase-never-enable-new-dtags.patch | ||
- patches/0019-qtbase-link-xcb-shm.patch | ||
- patches/0020-qtbase-Do-not-set-PKG_CONFIG_SYSROOT_DIR-when-sysroot-passe.patch | ||
#- patches/0021-qtbase-Add-debugging-to-debug-qtConfResolveLibs-and-also-to.patch | ||
- {{ patches }}/0001-qtbase-osx-xctest-check.patch | ||
- {{ patches }}/0002-qtbase-osx-allow-any-xcrun-in-PATH.patch | ||
- {{ patches }}/0003-qtbase-use-better-clang-optimize-size.patch | ||
- {{ patches }}/0004-qtbase-never-enable-new-dtags.patch | ||
- {{ patches }}/0005-qtbase-link-xcb-shm.patch | ||
- {{ patches }}/0006-qtbase-Do-not-set-PKG_CONFIG_SYSROOT_DIR-when-sysroo.patch | ||
- {{ patches }}/0007-qtbase-Disable-kTLSProtocol13-for-macOS-10.12.patch | ||
- {{ patches }}/0008-qtbase-Define-kIOSurfaceSuccess-if-it-is-not-defined.patch | ||
# - {{ patches }}/9999-qtbase-use-better-gcc-optimize-size.patch | ||
|
||
# qtwinextras | ||
- patches/0001-shobjidl-Fix-compile-guard-around-SHARDAPPIDINFOLINK.patch | ||
|
||
# qtwebengine | ||
- patches/0001-qtwebengine-allow-any-xcblah-in-PATH.patch | ||
- patches/0002-qtwebengine-find_sdk-respect-CONDA_BUILD_SYSROOT.patch | ||
|
@@ -40,6 +56,7 @@ source: | |
- patches/0012-qtwebengine-Ignore-glibc-2-17-min-version-check.patch | ||
- patches/0013-qtwebengine-missing_EVIOCGPROP.diff | ||
- patches/0014-qtwebengine-HAVE_SENDMMSG.diff | ||
|
||
# qtscript | ||
- patches/0001-qtscript-mark-cti_vm_throw-as-REFERENCED_FROM_ASM.patch | ||
|
||
|
@@ -61,7 +78,8 @@ source: | |
folder: opengl32sw # [win64] | ||
|
||
build: | ||
number: 0 | ||
number: {{ build_number }} | ||
string: {{ build_number }}{{ dev_nature }} | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Use the pkg hash here. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. For my debugging purposes I need to switch this depending on There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. i.e There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I meant use the pkg hash in addition. Something like There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. OK, cool |
||
skip: True # [win and vc != 14] | ||
detect_binary_files_with_prefix: true | ||
# QtWebEngine fails on Linux unless we merge | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,27 +1,28 @@ | ||
From 8005d91c1f098c87bf9c1e27a867a802b6e76b0f Mon Sep 17 00:00:00 2001 | ||
From 9fdd217d7bea7fbd55f26ce4a396b1d47de5ab27 Mon Sep 17 00:00:00 2001 | ||
From: Ray Donnelly <mingw.android@gmail.com> | ||
Date: Mon, 10 Dec 2018 14:51:15 -0600 | ||
Subject: [PATCH] qtbase: link xcb-shm | ||
Subject: [PATCH 5/6] qtbase: link xcb-shm | ||
|
||
Change-Id: I49300967cd92ed280a9a0c685e98a45c633118b6 | ||
--- | ||
src/gui/configure.json | 2 +- | ||
1 file changed, 1 insertion(+), 1 deletion(-) | ||
src/gui/configure.json | 4 ++-- | ||
1 file changed, 2 insertions(+), 2 deletions(-) | ||
|
||
diff --git a/qtbase/src/gui/configure.json b/qtbase/src/gui/configure.json | ||
index 0332631ec8..6b027edfc9 100644 | ||
index c51e3ceee3..2b08d84373 100644 | ||
--- a/qtbase/src/gui/configure.json | ||
+++ b/qtbase/src/gui/configure.json | ||
@@ -472,7 +472,7 @@ | ||
@@ -586,8 +586,8 @@ | ||
}, | ||
"headers": "xcb/xcb.h", | ||
"sources": [ | ||
- { "type": "pkgConfig", "args": "xcb >= 1.9" }, | ||
- "-lxcb" | ||
+ { "type": "pkgConfig", "args": "xcb >= 1.9 xcb-shm >= 1.9" }, | ||
+ "-lxcb -lxcb-shm" | ||
] | ||
}, | ||
"xcb_syslibs": { | ||
"xcb_icccm": { | ||
-- | ||
2.19.1 | ||
2.20.1 | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove these 2 lines
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We need these for Anaconda Distro, are they harmful?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, it's harmful for conda-forge as we are installing the SDK somewhere else.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
OK, I'm probably OK with aligning our location then. We do need to force 10.12 somehow though. What is the default ATM? Any clues how to force 10.12?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It is set to
$(xcode-select -p)/Platforms/MacOSX.platform/Developer/SDKs/MacOSX${MACOSX_DEPLOYMENT_TARGET}.sdk
which wade sense when we were using system compilers.Default is 10.9. If you set MACOSX_DEPLOYMENT_TARGET to 10.12 as you've done here in
recipe/conda_build_config.yaml
, the CI script will read that variable, download 10.12, extract it and automatically setCONDA_BUILD_SYSROOT
. So there's no need to setCONDA_BUILD_SYSROOT
in the recipe. JustMACOSX_DEPLOYMENT_TARGET
would do.