Closed
Bug 598237
Opened 14 years ago
Closed 10 years ago
Replace os.path functions with posixpath module
Categories
(Release Engineering :: General, enhancement, P5)
Release Engineering
General
Tracking
(Not tracked)
RESOLVED
WONTFIX
People
(Reporter: jhford, Unassigned)
Details
(Whiteboard: [kanban:engops:https://mozilla.kanbanize.com/ctrl_board/6/2995] [buildbotcustom][cleanup])
Instead of using os.path which has a dependency on the runtime platform of the master for generating buildbot step arguments, we should explicitly use one of the platform path modules. 'posixpath' seems the most logical. This isn't too important, but seems like good hygiene as our configurations rely on the posix behaviour. jhford-wifi:~ jhford$ python Python 2.6.1 (r261:67515, Feb 11 2010, 00:51:29) [GCC 4.2.1 (Apple Inc. build 5646)] on darwin Type "help", "copyright", "credits" or "license" for more information. >>> import os.path >>> import posixpath >>> os.path is posixpath True >>>
Updated•12 years ago
|
Component: Release Engineering → Release Engineering: Automation (General)
QA Contact: release → catlee
Whiteboard: [buildbotcustom][cleanup]
Assignee | ||
Updated•11 years ago
|
Product: mozilla.org → Release Engineering
Updated•10 years ago
|
Whiteboard: [buildbotcustom][cleanup] → [kanban:engops:https://mozilla.kanbanize.com/ctrl_board/6/2985] [buildbotcustom][cleanup]
Updated•10 years ago
|
Whiteboard: [kanban:engops:https://mozilla.kanbanize.com/ctrl_board/6/2985] [buildbotcustom][cleanup] → [kanban:engops:https://mozilla.kanbanize.com/ctrl_board/6/2990] [buildbotcustom][cleanup]
Updated•10 years ago
|
Whiteboard: [kanban:engops:https://mozilla.kanbanize.com/ctrl_board/6/2990] [buildbotcustom][cleanup] → [kanban:engops:https://mozilla.kanbanize.com/ctrl_board/6/2995] [buildbotcustom][cleanup]
Comment 1•10 years ago
|
||
I can't imagine us doing this for the sake of doing it. We should do it as we refactor/replace this code though.
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → WONTFIX
Assignee | ||
Updated•6 years ago
|
Component: General Automation → General
You need to log in
before you can comment on or make changes to this bug.
Description
•