diff options
| author | Ross Burton <ross@burtonini.com> | 2021-04-26 11:27:51 +0800 |
|---|---|---|
| committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2021-04-27 15:22:58 +0100 |
| commit | 3f8758b54dc45f16cb28689271a2f580917e4078 (patch) | |
| tree | c41455c902dd4a4bbf7ef0f83976bbd66a98c92c | |
| parent | b7cba05f8267182ac553f552a5f57a91fe5e7eee (diff) | |
| download | poky-3f8758b54dc45f16cb28689271a2f580917e4078.tar.gz | |
bitbake: bitbake-server: ensure server timeout is a float
bitbake-server is spawned by process.py and passes the arguments it is
given to ProcessServer. There's some type confusion here:
bitbake-server is called with a string representation of the timeout,
which may be None. If the timeout is not set, pass 0 instead of None.
Inside bitbake-server a ProcessServer is created which expects the
timeout to be a float not a string, so always float() the value.
[ YOCTO #14350 ]
(Bitbake rev: f2cfb9f6710808ea37aecb6c34c62f92191e1d4b)
Signed-off-by: Ross Burton <ross.burton@arm.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
(cherry picked from commit c93ae1f861208f6d39fd15c84fbcd0e2b54331f5)
Signed-off-by: Anuj Mittal <anuj.mittal@intel.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
| -rwxr-xr-x | bitbake/bin/bitbake-server | 2 | ||||
| -rw-r--r-- | bitbake/lib/bb/server/process.py | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/bitbake/bin/bitbake-server b/bitbake/bin/bitbake-server index ffbc7894ef..65796be747 100755 --- a/bitbake/bin/bitbake-server +++ b/bitbake/bin/bitbake-server | |||
| @@ -26,7 +26,7 @@ readypipeinfd = int(sys.argv[3]) | |||
| 26 | logfile = sys.argv[4] | 26 | logfile = sys.argv[4] |
| 27 | lockname = sys.argv[5] | 27 | lockname = sys.argv[5] |
| 28 | sockname = sys.argv[6] | 28 | sockname = sys.argv[6] |
| 29 | timeout = sys.argv[7] | 29 | timeout = float(sys.argv[7]) |
| 30 | xmlrpcinterface = (sys.argv[8], int(sys.argv[9])) | 30 | xmlrpcinterface = (sys.argv[8], int(sys.argv[9])) |
| 31 | if xmlrpcinterface[0] == "None": | 31 | if xmlrpcinterface[0] == "None": |
| 32 | xmlrpcinterface = (None, xmlrpcinterface[1]) | 32 | xmlrpcinterface = (None, xmlrpcinterface[1]) |
diff --git a/bitbake/lib/bb/server/process.py b/bitbake/lib/bb/server/process.py index b27b4aefe0..3e99bcef8f 100644 --- a/bitbake/lib/bb/server/process.py +++ b/bitbake/lib/bb/server/process.py | |||
| @@ -509,7 +509,7 @@ class BitBakeServer(object): | |||
| 509 | os.set_inheritable(self.bitbake_lock.fileno(), True) | 509 | os.set_inheritable(self.bitbake_lock.fileno(), True) |
| 510 | os.set_inheritable(self.readypipein, True) | 510 | os.set_inheritable(self.readypipein, True) |
| 511 | serverscript = os.path.realpath(os.path.dirname(__file__) + "/../../../bin/bitbake-server") | 511 | serverscript = os.path.realpath(os.path.dirname(__file__) + "/../../../bin/bitbake-server") |
| 512 | os.execl(sys.executable, "bitbake-server", serverscript, "decafbad", str(self.bitbake_lock.fileno()), str(self.readypipein), self.logfile, self.bitbake_lock.name, self.sockname, str(self.server_timeout), str(self.xmlrpcinterface[0]), str(self.xmlrpcinterface[1])) | 512 | os.execl(sys.executable, "bitbake-server", serverscript, "decafbad", str(self.bitbake_lock.fileno()), str(self.readypipein), self.logfile, self.bitbake_lock.name, self.sockname, str(self.server_timeout or 0), str(self.xmlrpcinterface[0]), str(self.xmlrpcinterface[1])) |
| 513 | 513 | ||
| 514 | def execServer(lockfd, readypipeinfd, lockname, sockname, server_timeout, xmlrpcinterface): | 514 | def execServer(lockfd, readypipeinfd, lockname, sockname, server_timeout, xmlrpcinterface): |
| 515 | 515 | ||
