diff options
author | Richard Purdie <richard.purdie@linuxfoundation.org> | 2013-11-19 14:32:08 +0000 |
---|---|---|
committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2013-11-22 12:03:02 +0000 |
commit | 9d7f8e2a206f8266fd0766b6161dbd1bf6b787a7 (patch) | |
tree | 2002648fd17558c7d895d672aa65fa46f4b82250 /bitbake/lib/bb/fetch2/gitsm.py | |
parent | 4acc7322a2ada941b803e1aee23d84351888eef0 (diff) | |
download | poky-9d7f8e2a206f8266fd0766b6161dbd1bf6b787a7.tar.gz |
bitbake: fetch2: Stop passing around the pointless url parameter
There is no good reason to keep passing around the url parameter when
its contained within urldata (ud). This is left around due to
legacy reasons, some functions take it, some don't and its time
to cleanup.
This is fetcher internal API, there are a tiny number of external users
of the internal API (buildhistory and distrodata) which can be fixed up
after this change.
(Bitbake rev: 6a48474de9505a3700863f31839a7c53c5e18a8d)
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Diffstat (limited to 'bitbake/lib/bb/fetch2/gitsm.py')
-rw-r--r-- | bitbake/lib/bb/fetch2/gitsm.py | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/bitbake/lib/bb/fetch2/gitsm.py b/bitbake/lib/bb/fetch2/gitsm.py index 572b637c9a..4093f6e846 100644 --- a/bitbake/lib/bb/fetch2/gitsm.py +++ b/bitbake/lib/bb/fetch2/gitsm.py | |||
@@ -27,7 +27,7 @@ from bb.fetch2 import runfetchcmd | |||
27 | from bb.fetch2 import logger | 27 | from bb.fetch2 import logger |
28 | 28 | ||
29 | class GitSM(Git): | 29 | class GitSM(Git): |
30 | def supports(self, url, ud, d): | 30 | def supports(self, ud, d): |
31 | """ | 31 | """ |
32 | Check to see if a given url can be fetched with git. | 32 | Check to see if a given url can be fetched with git. |
33 | """ | 33 | """ |
@@ -42,7 +42,7 @@ class GitSM(Git): | |||
42 | pass | 42 | pass |
43 | return False | 43 | return False |
44 | 44 | ||
45 | def update_submodules(self, u, ud, d): | 45 | def update_submodules(self, ud, d): |
46 | # We have to convert bare -> full repo, do the submodule bit, then convert back | 46 | # We have to convert bare -> full repo, do the submodule bit, then convert back |
47 | tmpclonedir = ud.clonedir + ".tmp" | 47 | tmpclonedir = ud.clonedir + ".tmp" |
48 | gitdir = tmpclonedir + os.sep + ".git" | 48 | gitdir = tmpclonedir + os.sep + ".git" |
@@ -58,13 +58,13 @@ class GitSM(Git): | |||
58 | os.rename(gitdir, ud.clonedir,) | 58 | os.rename(gitdir, ud.clonedir,) |
59 | bb.utils.remove(tmpclonedir, True) | 59 | bb.utils.remove(tmpclonedir, True) |
60 | 60 | ||
61 | def download(self, loc, ud, d): | 61 | def download(self, ud, d): |
62 | Git.download(self, loc, ud, d) | 62 | Git.download(self, ud, d) |
63 | 63 | ||
64 | os.chdir(ud.clonedir) | 64 | os.chdir(ud.clonedir) |
65 | submodules = self.uses_submodules(ud, d) | 65 | submodules = self.uses_submodules(ud, d) |
66 | if submodules: | 66 | if submodules: |
67 | self.update_submodules(loc, ud, d) | 67 | self.update_submodules(ud, d) |
68 | 68 | ||
69 | def unpack(self, ud, destdir, d): | 69 | def unpack(self, ud, destdir, d): |
70 | Git.unpack(self, ud, destdir, d) | 70 | Git.unpack(self, ud, destdir, d) |