fix pylint warning: build/toolchain/mac/linker_driver.py

fix pylint warning: build/toolchain/mac/linker_driver.py

W:134, 3: Unused variable 'head' (unused-variable)
W:159, 3: Unused variable 'head' (unused-variable)

Committed: https://crrev.com/f636eadf2f2096579005f581cc87963429cf4444
Cr-Commit-Position: refs/heads/master@{#423410}

[Yoshisato Yanagisawa, 2016-10-05 09:38:15]

yyanagisawa@chromium.org changed reviewers:
+ dpranke@chromium.org, rsesek@chromium.org

[Yoshisato Yanagisawa, 2016-10-05 09:38:41]

Description was changed from

==========
fix pylint error: build/toolchain/mac/linker_driver.py

W:134, 3: Unused variable 'head' (unused-variable)
W:159, 3: Unused variable 'head' (unused-variable)
==========

to

==========
fix pylint warning: build/toolchain/mac/linker_driver.py

W:134, 3: Unused variable 'head' (unused-variable)
W:159, 3: Unused variable 'head' (unused-variable)
==========

[Robert Sesek, 2016-10-05 20:29:27]

lgtm

[Dirk Pranke, 2016-10-06 01:28:15]

lgtm w/ nit.

https://codereview.chromium.org/2396723003/diff/1/build/toolchain/mac/linker_...
File build/toolchain/mac/linker_driver.py (right):

https://codereview.chromium.org/2396723003/diff/1/build/toolchain/mac/linker_...
build/toolchain/mac/linker_driver.py:133: out = os.path.split(linker_out)[1]
Just use `os.path.basename(linker_out)` here and below instead.

[Yoshisato Yanagisawa, 2016-10-06 01:37:31]

https://codereview.chromium.org/2396723003/diff/1/build/toolchain/mac/linker_...
File build/toolchain/mac/linker_driver.py (right):

https://codereview.chromium.org/2396723003/diff/1/build/toolchain/mac/linker_...
build/toolchain/mac/linker_driver.py:133: out = os.path.split(linker_out)[1]
On 2016/10/06 01:28:15, Dirk Pranke (slow) wrote:
> Just use `os.path.basename(linker_out)` here and below instead.

Done.

[Yoshisato Yanagisawa, 2016-10-06 01:37:44]

The CQ bit was checked by yyanagisawa@chromium.org

[Yoshisato Yanagisawa, 2016-10-06 01:37:44]

The patchset sent to the CQ was uploaded after l-g-t-m from rsesek@chromium.org,
dpranke@chromium.org
Link to the patchset: https://codereview.chromium.org/2396723003/#ps20001
(title: "use os.path.basename instead of os.path.split")

[commit-bot: I haz the power, 2016-10-06 01:38:22]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-10-06 02:53:30]

Description was changed from

==========
fix pylint warning: build/toolchain/mac/linker_driver.py

W:134, 3: Unused variable 'head' (unused-variable)
W:159, 3: Unused variable 'head' (unused-variable)
==========

to

==========
fix pylint warning: build/toolchain/mac/linker_driver.py

W:134, 3: Unused variable 'head' (unused-variable)
W:159, 3: Unused variable 'head' (unused-variable)
==========

[commit-bot: I haz the power, 2016-10-06 02:53:31]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-10-06 02:56:25]

Description was changed from

==========
fix pylint warning: build/toolchain/mac/linker_driver.py

W:134, 3: Unused variable 'head' (unused-variable)
W:159, 3: Unused variable 'head' (unused-variable)
==========

to

==========
fix pylint warning: build/toolchain/mac/linker_driver.py

W:134, 3: Unused variable 'head' (unused-variable)
W:159, 3: Unused variable 'head' (unused-variable)

Committed: https://crrev.com/f636eadf2f2096579005f581cc87963429cf4444
Cr-Commit-Position: refs/heads/master@{#423410}
==========

[commit-bot: I haz the power, 2016-10-06 02:56:26]

Patchset 2 (id:??) landed as
https://crrev.com/f636eadf2f2096579005f581cc87963429cf4444
Cr-Commit-Position: refs/heads/master@{#423410}