[gentoo-dev] [PATCH 2/2] Allow user mangle distfiles' ${EGIT_DIR} after actual git fetch.
EGIT_REPO_URI=https://github.com/ghc/ghc.git;
requires user to run './sync-all fetch / ./sync-all pull'
after actual 'git pull', which fetches 20 more repos for code changes.
Upstream does not use submodules.
The patch injects user's callback right before 'git-2_move_source'.
Currently I abuse 'git-2_gc':
Original ebuild:
https://github.com/gentoo-haskell/gentoo-haskell/blob/master/dev-lang/ghc/ghc-.ebuild#L180
Signed-off-by: Sergei Trofimovich sly...@gentoo.org
---
git-2.eclass | 24
1 file changed, 24 insertions(+)
diff --git a/git-2.eclass b/git-2.eclass
index 1a96978..1bacef5 100644
--- a/git-2.eclass
+++ b/git-2.eclass
@@ -569,6 +569,29 @@ git-2_cleanup() {
unset EGIT_LOCAL_NONBARE
}
+
+# @FUNCTION: git-2_fetch_user
+# @DESCRIPTION:
+# User-overridable callback allow user to update
+# sources in ${EGIT_DIR} (current location).
+# Does nothing by default
+git-2_fetch_user() {
+ :
+}
+
+# @FUNCTION: git-2_post_fetch
+# @INTERNAL
+# Internal function calling user's callback
+# when ${EGIT_DIR} needs more actions, than
+# simple fetch.
+git-2_post_fetch() {
+ debug-print-function ${FUNCNAME} $@
+
+ pushd ${EGIT_DIR} /dev/null
+ git-2_fetch_user
+ popd /dev/null
+}
+
# @FUNCTION: git-2_src_unpack
# @DESCRIPTION:
# Default git src_unpack function.
@@ -581,6 +604,7 @@ git-2_src_unpack() {
git-2_fetch $@
git-2_gc
git-2_submodules
+ git-2_post_fetch
git-2_move_source
git-2_branch
git-2_bootstrap
--
1.8.0
Re: [gentoo-dev] [PATCH 2/2] Allow user mangle distfiles' ${EGIT_DIR} after actual git fetch.
On Tue, 27 Nov 2012 22:22:20 +0300
Sergei Trofimovich sly...@gentoo.org wrote:
EGIT_REPO_URI=https://github.com/ghc/ghc.git;
requires user to run './sync-all fetch / ./sync-all pull'
after actual 'git pull', which fetches 20 more repos for code changes.
Upstream does not use submodules.
The patch injects user's callback right before 'git-2_move_source'.
Currently I abuse 'git-2_gc':
Original ebuild:
https://github.com/gentoo-haskell/gentoo-haskell/blob/master/dev-lang/ghc/ghc-.ebuild#L180
Signed-off-by: Sergei Trofimovich sly...@gentoo.org
---
git-2.eclass | 24
1 file changed, 24 insertions(+)
diff --git a/git-2.eclass b/git-2.eclass
index 1a96978..1bacef5 100644
--- a/git-2.eclass
+++ b/git-2.eclass
@@ -569,6 +569,29 @@ git-2_cleanup() {
unset EGIT_LOCAL_NONBARE
}
+
+# @FUNCTION: git-2_fetch_user
+# @DESCRIPTION:
+# User-overridable callback allow user to update
+# sources in ${EGIT_DIR} (current location).
+# Does nothing by default
+git-2_fetch_user() {
+ :
+}
+
+# @FUNCTION: git-2_post_fetch
+# @INTERNAL
+# Internal function calling user's callback
+# when ${EGIT_DIR} needs more actions, than
+# simple fetch.
+git-2_post_fetch() {
+ debug-print-function ${FUNCNAME} $@
+
+ pushd ${EGIT_DIR} /dev/null
+ git-2_fetch_user
+ popd /dev/null
+}
+
# @FUNCTION: git-2_src_unpack
# @DESCRIPTION:
# Default git src_unpack function.
@@ -581,6 +604,7 @@ git-2_src_unpack() {
git-2_fetch $@
git-2_gc
git-2_submodules
+ git-2_post_fetch
git-2_move_source
git-2_branch
git-2_bootstrap
That's not something that can or should be supported. This will confuse
at least smart-live-rebuild.
--
Best regards,
Michał Górny
signature.asc
Description: PGP signature
Re: [gentoo-dev] [PATCH 2/2] Allow user mangle distfiles' ${EGIT_DIR} after actual git fetch.
This is bad idea. It breaks live rebuild and other stuff. You should just
clone each repo yourself, see how i did in libreoffice ebuild
Dne 27.11.2012 20:28 Sergei Trofimovich sly...@gentoo.org napsal(a):
EGIT_REPO_URI=https://github.com/ghc/ghc.git;
requires user to run './sync-all fetch / ./sync-all pull'
after actual 'git pull', which fetches 20 more repos for code changes.
Upstream does not use submodules.
The patch injects user's callback right before 'git-2_move_source'.
Currently I abuse 'git-2_gc':
Original ebuild:
https://github.com/gentoo-haskell/gentoo-haskell/blob/master/dev-lang/ghc/ghc-.ebuild#L180
Signed-off-by: Sergei Trofimovich sly...@gentoo.org
---
git-2.eclass | 24
1 file changed, 24 insertions(+)
diff --git a/git-2.eclass b/git-2.eclass
index 1a96978..1bacef5 100644
--- a/git-2.eclass
+++ b/git-2.eclass
@@ -569,6 +569,29 @@ git-2_cleanup() {
unset EGIT_LOCAL_NONBARE
}
+
+# @FUNCTION: git-2_fetch_user
+# @DESCRIPTION:
+# User-overridable callback allow user to update
+# sources in ${EGIT_DIR} (current location).
+# Does nothing by default
+git-2_fetch_user() {
+ :
+}
+
+# @FUNCTION: git-2_post_fetch
+# @INTERNAL
+# Internal function calling user's callback
+# when ${EGIT_DIR} needs more actions, than
+# simple fetch.
+git-2_post_fetch() {
+ debug-print-function ${FUNCNAME} $@
+
+ pushd ${EGIT_DIR} /dev/null
+ git-2_fetch_user
+ popd /dev/null
+}
+
# @FUNCTION: git-2_src_unpack
# @DESCRIPTION:
# Default git src_unpack function.
@@ -581,6 +604,7 @@ git-2_src_unpack() {
git-2_fetch $@
git-2_gc
git-2_submodules
+ git-2_post_fetch
git-2_move_source
git-2_branch
git-2_bootstrap
--
1.8.0
Re: [gentoo-dev] [PATCH 2/2] Allow user mangle distfiles' ${EGIT_DIR} after actual git fetch.
On Tue, 27 Nov 2012 21:46:06 +0100 Tomáš Chvátal tomas.chva...@gmail.com wrote: This is bad idea. It breaks live rebuild and other stuff. You should just clone each repo yourself, see how i did in libreoffice ebuild How does it break live rebuild? I'm not sure I understand what exactly you mean. Does libreoffice export all the info for each submodule the way smart-live-rebuild can pick it? Dne 27.11.2012 20:28 Sergei Trofimovich sly...@gentoo.org napsal(a): EGIT_REPO_URI=https://github.com/ghc/ghc.git; requires user to run './sync-all fetch / ./sync-all pull' after actual 'git pull', which fetches 20 more repos for code changes. Upstream does not use submodules. The patch injects user's callback right before 'git-2_move_source'. Currently I abuse 'git-2_gc': Original ebuild: https://github.com/gentoo-haskell/gentoo-haskell/blob/master/dev-lang/ghc/ghc-.ebuild#L180 -- Sergei signature.asc Description: PGP signature
