 package/qt5/qt5.mk                   |  1 +
 package/qt5/qt5webkit/qt5webkit.hash |  4 ++--
 package/qt5/qt5webkit/qt5webkit.mk   | 14 +++-----------
 3 files changed, 6 insertions(+), 13 deletions(-)

package/qt5/qt5.mk +1 -0

index d25f663662ee..e937e234b596 100644
@@ -1,6 +1,7 @@
 QT5_VERSION_MAJOR = 5.6
 QT5_VERSION = $(QT5_VERSION_MAJOR).2
 QT5_SITE = http://download.qt.io/official_releases/qt/$(QT5_VERSION_MAJOR)/$(QT5_VERSION)/submodules
+QT5_SNAPSHOTS_SITE = http://download.qt.io/snapshots/qt/$(QT5_VERSION_MAJOR)/$(QT5_VERSION)/latest_src/submodules
 include $(sort $(wildcard package/qt5/*/*.mk))
 
 define QT5_LA_PRL_FILES_FIXUP

package/qt5/qt5webkit/qt5webkit.hash +2 -2

index 96b8bdd6909a..309f776b3ff3 100644
@@ -1,2 +1,2 @@
-# locally computed
-sha256 bdd659573e7e75cd4ad57b7160a7353d98d21a6fef06e4fb9e114a5b1f1e9dab qt5webkit-b35917bcb44d7f200af0f4ac68a126fa0aa8d93d.tar.gz
+# Hash from: http://download.qt.io/snapshots/qt/5.6/5.6.2/latest_src/submodules/qtwebkit-opensource-src-5.6.2.tar.xz.mirrorlist
+sha256 528a6b8b1c5095367b26e8ce4f3a46bb739e2e9913ff4dfc6ef58a04fcd73966 qtwebkit-opensource-src-5.6.2.tar.xz

package/qt5/qt5webkit/qt5webkit.mk +3 ~~-11~~

index 378cdf78573e..980d2aff0167 100644
@@ -4,10 +4,9 @@
 #
 ################################################################################
 
-QT5WEBKIT_VERSION = b35917bcb44d7f200af0f4ac68a126fa0aa8d93d
-# Using GitHub since it supports downloading tarballs from random commits.
-# The http://code.qt.io/cgit/qt/qtwebkit.git/ repo doesn't allow to do so.
-QT5WEBKIT_SITE = $(call github,qtproject,qtwebkit,$(QT5WEBKIT_VERSION))
+QT5WEBKIT_VERSION = $(QT5_VERSION)
+QT5WEBKIT_SITE = $(QT5_SNAPSHOTS_SITE)
+QT5WEBKIT_SOURCE = qtwebkit-opensource-src-$(QT5WEBKIT_VERSION).tar.xz
 QT5WEBKIT_DEPENDENCIES = \
 	host-bison host-flex host-gperf host-python host-ruby \
 	qt5base sqlite

@@ -43,14 +42,7 @@ define QT5WEBKIT_PYTHON2_SYMLINK
 endef
 QT5WEBKIT_PRE_CONFIGURE_HOOKS += QT5WEBKIT_PYTHON2_SYMLINK
 
-# Since we get the source from git, generated header files are not included.
-# qmake detects that header file generation (using the syncqt tool) must be
-# done based on the existence of a .git directory (cfr. the git_build config
-# option which is set in qt_build_paths.prf).
-# So, to make sure that qmake detects that header files must be generated,
-# create an empty .git directory.
 define QT5WEBKIT_CONFIGURE_CMDS
-	mkdir -p $(@D)/.git
 	(cd $(@D); $(TARGET_MAKE_ENV) $(QT5WEBKIT_ENV) $(HOST_DIR)/usr/bin/qmake)
 endef

Signature · 1 line

2.7.4

Julien CORJON

2016-11-15 14:07 UTC #1 lore raw

PATCH v2 1/1 qt5webkit: Get sources from Qt-5-unofficial-builds

Alexey,

I like the idea of fetching obsolete submodules from Qt hosting instead of external github. But, as url say it's snapshot, we will probably have to change hash file more often... Maybe we will have to get rid of hash files for theses obsolete submodules.

Still I'm much more for this solution so :

Reviewed-by Julien Corjon <[email protected]>

Regards,

Julien Corjon

Le 09/11/2016 ? 14:59, Alexey Brodkin a ?crit :

Quoted 85 lines

Instead of messing with git we may get tarballs of obsolete but still existing submodules as we used to but from a bit different location.

It turned out Qt people still create packages for obsolete submodules for so-called "unofficial builds", see [1].

[1] https://wiki.qt.io/Qt-5-unofficial-builds

Signed-off-by Alexey Brodkin <[email protected]>

Cc Yegor Yefremov <[email protected]>

Cc Thomas Petazzoni <[email protected]>

Cc Peter Seiderer <[email protected]>

Cc "Yann E. MORIN" <[email protected]>

Cc Jerzy Grzegorek <[email protected]>

Cc Peter Korsgaard <[email protected]>

Cc Julien Corjon <[email protected]>

Cc Gary Bisson <[email protected]>

Cc J?r?me Pouiller <[email protected]>

---

Changes v1 -> v2:
* Fixes copy-pasted QT5WEBKIT_VERSION
(was QT5WEBKIT_EXAMPLES_VERSION)
Thanks to J?r?me!

 package/qt5/qt5.mk                   |  1 +
 package/qt5/qt5webkit/qt5webkit.hash |  4 ++--
 package/qt5/qt5webkit/qt5webkit.mk   | 14 +++-----------
 3 files changed, 6 insertions(+), 13 deletions(-)

package/qt5/qt5.mk +1 -0

index d25f663662ee..e937e234b596 100644
@@ -1,6 +1,7 @@
 QT5_VERSION_MAJOR = 5.6
 QT5_VERSION = $(QT5_VERSION_MAJOR).2
 QT5_SITE = http://download.qt.io/official_releases/qt/$(QT5_VERSION_MAJOR)/$(QT5_VERSION)/submodules
+QT5_SNAPSHOTS_SITE = http://download.qt.io/snapshots/qt/$(QT5_VERSION_MAJOR)/$(QT5_VERSION)/latest_src/submodules
 include $(sort $(wildcard package/qt5/*/*.mk))
 
 define QT5_LA_PRL_FILES_FIXUP

package/qt5/qt5webkit/qt5webkit.hash +2 -2

index 96b8bdd6909a..309f776b3ff3 100644
@@ -1,2 +1,2 @@
-# locally computed
-sha256 bdd659573e7e75cd4ad57b7160a7353d98d21a6fef06e4fb9e114a5b1f1e9dab qt5webkit-b35917bcb44d7f200af0f4ac68a126fa0aa8d93d.tar.gz
+# Hash from: http://download.qt.io/snapshots/qt/5.6/5.6.2/latest_src/submodules/qtwebkit-opensource-src-5.6.2.tar.xz.mirrorlist
+sha256 528a6b8b1c5095367b26e8ce4f3a46bb739e2e9913ff4dfc6ef58a04fcd73966 qtwebkit-opensource-src-5.6.2.tar.xz

package/qt5/qt5webkit/qt5webkit.mk +3 ~~-11~~

index 378cdf78573e..980d2aff0167 100644
@@ -4,10 +4,9 @@
 #
 ################################################################################
 
-QT5WEBKIT_VERSION = b35917bcb44d7f200af0f4ac68a126fa0aa8d93d
-# Using GitHub since it supports downloading tarballs from random commits.
-# The http://code.qt.io/cgit/qt/qtwebkit.git/ repo doesn't allow to do so.
-QT5WEBKIT_SITE = $(call github,qtproject,qtwebkit,$(QT5WEBKIT_VERSION))
+QT5WEBKIT_VERSION = $(QT5_VERSION)
+QT5WEBKIT_SITE = $(QT5_SNAPSHOTS_SITE)
+QT5WEBKIT_SOURCE = qtwebkit-opensource-src-$(QT5WEBKIT_VERSION).tar.xz
 QT5WEBKIT_DEPENDENCIES = \
 	host-bison host-flex host-gperf host-python host-ruby \
 	qt5base sqlite

@@ -43,14 +42,7 @@ define QT5WEBKIT_PYTHON2_SYMLINK
 endef
 QT5WEBKIT_PRE_CONFIGURE_HOOKS += QT5WEBKIT_PYTHON2_SYMLINK
 
-# Since we get the source from git, generated header files are not included.
-# qmake detects that header file generation (using the syncqt tool) must be
-# done based on the existence of a .git directory (cfr. the git_build config
-# option which is set in qt_build_paths.prf).
-# So, to make sure that qmake detects that header files must be generated,
-# create an empty .git directory.
 define QT5WEBKIT_CONFIGURE_CMDS
-	mkdir -p $(@D)/.git
 	(cd $(@D); $(TARGET_MAKE_ENV) $(QT5WEBKIT_ENV) $(HOST_DIR)/usr/bin/qmake)
 endef

Arnout Vandecappelle

2016-11-15 21:20 UTC #2 lore raw

PATCH v2 1/1 qt5webkit: Get sources from Qt-5-unofficial-builds

Quoted 6 lines · Julien CORJON

Alexey,
I like the idea of fetching obsolete submodules from Qt hosting instead of external github. But, as url say it's snapshot, we will probably have to change hash file more often... Maybe we will have to get rid of hash files for theses obsolete submodules.

Since the tarballs are actual versioned tarballs, I do hope that they don't
just change randomly...

However, there is a risk that the tarballs just disappear after some time. But
we have sources.buildroot.org for that.

Regards,
Arnout

Quoted 3 lines

Still I'm much more for this solution so :
Reviewed-by Julien Corjon <[email protected]>

Signature · 6 lines

Arnout Vandecappelle                          arnout at mind be
Senior Embedded Software Architect            +32-16-286500
Essensium/Mind                                http://www.mind.be
G.Geenslaan 9, 3001 Leuven, Belgium           BE 872 984 063 RPR Leuven
LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
GPG fingerprint:  7493 020B C7E3 8618 8DEC 222C 82EB F404 F9AC 0DDF

Andreas Naumann

2016-11-16 07:53 UTC #3 lore raw

PATCH v2 1/1 qt5webkit: Get sources from Qt-5-unofficial-builds

Am 15.11.2016 um 22:20 schrieb Arnout Vandecappelle:

Quoted 12 lines · Julien CORJON

Quoted 6 lines
Alexey,
I like the idea of fetching obsolete submodules from Qt hosting instead of external github. But, as url say it's snapshot, we will probably have to change hash file more often... Maybe we will have to get rid of hash files for theses obsolete submodules.
Since the tarballs are actual versioned tarballs, I do hope that they don't
just change randomly...

So my experience was that the tarballs in 5.8.0-beta/latest_src/submodules did actually change until it was properly released. That was the reason i removed the hashes in my bump series (after having refreshed them initially).
I guess the .../<number>/submodule snapshots are probably more constant during the development phase, but I have no objection to leave the patch as is.

Quoted 11 lines

However, there is a risk that the tarballs just disappear after some time. But
we have sources.buildroot.org for that.
Regards,
Arnout
Quoted 3 lines
Still I'm much more for this solution so :
Reviewed-by Julien Corjon <[email protected]>

Alexey Brodkin

2016-11-16 08:14 UTC #4 lore raw

PATCH v2 1/1 qt5webkit: Get sources from Qt-5-unofficial-builds

Hi?Andreas,

Quoted 16 lines · Andreas Naumann

Am 15.11.2016 um 22:20 schrieb Arnout Vandecappelle:
Quoted 14 lines · Julien CORJON
Quoted 7 lines
Alexey,
I like the idea of fetching obsolete submodules from Qt hosting instead of external github. But, as url say it's snapshot, we will probably have to change hash file more often... Maybe we will have to get rid of hash files for theses obsolete submodules.
?Since the tarballs are actual versioned tarballs, I do hope that they don't just change randomly...

Quoted 7 lines

So my experience was that the tarballs in? 5.8.0-beta/latest_src/submodules did actually change until it was? properly released. That was the reason i removed the hashes in my bump? series (after having refreshed them initially).
I guess the .../<number>/submodule snapshots are probably more constant? during the development phase, but I have no objection to leave the patch? as is.

Not really sure. In "numbered" folders I only see pre-built stuff like:
.exe and .dmg

So from what I may see the only source of sources is 5.8/5.8.0-beta/latest_src/.

-Alexey

Andreas Naumann

2016-11-16 08:21 UTC #5 lore raw

PATCH v2 1/1 qt5webkit: Get sources from Qt-5-unofficial-builds

Hi Alexey,

Am 16.11.2016 um 09:14 schrieb Alexey Brodkin:

Quoted 27 lines

Hi Andreas,
Quoted 16 lines · Andreas Naumann
Am 15.11.2016 um 22:20 schrieb Arnout Vandecappelle:
Quoted 14 lines · Julien CORJON
Quoted 7 lines
Alexey,
I like the idea of fetching obsolete submodules from Qt hosting instead of external github. But, as url say it's snapshot, we will probably have to change hash file more often... Maybe we will have to get rid of hash files for theses obsolete submodules.
Since the tarballs are actual versioned tarballs, I do hope that they don't
just change randomly...
Quoted 7 lines
So my experience was that the tarballs in 5.8.0-beta/latest_src/submodules did actually change until it was properly released. That was the reason i removed the hashes in my bump series (after having refreshed them initially).
I guess the .../<number>/submodule snapshots are probably more constant during the development phase, but I have no objection to leave the patch as is.

Quoted 2 lines

Not really sure. In "numbered" folders I only see pre-built stuff like:
.exe and .dmg

Sorry, true. I forgot about the content.

Quoted 10 lines

So from what I may see the only source of sources is 5.8/5.8.0-beta/latest_src/.
-Alexey
_______________________________________________
buildroot mailing list
buildroot at busybox.net
http://lists.busybox.net/mailman/listinfo/buildroot

Thomas Petazzoni

Applied

2016-11-15 22:41 UTC #6 lore raw

PATCH v2 1/1 qt5webkit: Get sources from Qt-5-unofficial-builds

In reply to Alexey Brodkin

Hello,

Quoted 18 lines · Alexey Brodkin

Instead of messing with git we may get tarballs of obsolete but still existing submodules as we used to but from a bit different location.
It turned out Qt people still create packages for obsolete submodules for so-called "unofficial builds", see [1].
[1] https://wiki.qt.io/Qt-5-unofficial-builds
Signed-off-by Alexey Brodkin <[email protected]>
Cc Yegor Yefremov <[email protected]>
Cc Thomas Petazzoni <[email protected]>
Cc Peter Seiderer <[email protected]>
Cc "Yann E. MORIN" <[email protected]>
Cc Jerzy Grzegorek <[email protected]>
Cc Peter Korsgaard <[email protected]>
Cc Julien Corjon <[email protected]>
Cc Gary Bisson <[email protected]>
Cc J?r?me Pouiller <[email protected]>

Quoted 1 line

---

Applied to next, thanks.

Thomas

Signature · 3 lines

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