Skip to content

feat: env-var for additional interpreter args in bootstrap stage 1#2654

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
rickeylev merged 7 commits into from
Mar 17, 2025
Merged

feat: env-var for additional interpreter args in bootstrap stage 1 #2654

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
827d77b
feat: env-var for additional interpreter args in bootstrap
andponlin-canva Mar 8, 2025
9aa5eb6
feat: env-var for additional interpreter args in bootstrap - add unse…
andponlin-canva Mar 10, 2025
14c2bfc
feat: env-var for additional interpreter args in bootstrap stage 1 --…
andponlin-canva Mar 10, 2025
d7ecc3b
feat: env-var for additional interpreter args in bootstrap stage 1 --…
andponlin-canva Mar 12, 2025
c02f3d8
Merge branch 'main' into bootstrp-additional-interp-args
aignas Mar 17, 2025
6026846
add docs to cross ref interpreter_args and env var
rickeylev Mar 17, 2025
df484a1
xref changelog
rickeylev Mar 17, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 3 additions & 0 deletions CHANGELOG.md
View file
Open in desktop
Original file line numberDiff line numberDiff line change
Expand Up@@ -94,6 +94,9 @@ Unreleased changes template.
* (rules) APIs for creating custom rules based on the core py_binary, py_test,
and py_library rules
([#1647](https://github.com/bazelbuild/rules_python/issues/1647))
* (rules) Added env-var to allow additional interpreter args for stage1 bootstrap.
See {obj}`RULES_PYTHON_ADDITIONAL_INTERPRETER_ARGS` environment variable.
Only applicable for {obj}`--bootstrap_impl=script`.
* (rules) Added {obj}`interpreter_args` attribute to `py_binary` and `py_test`,
which allows pass arguments to the interpreter before the regular args.

Expand Down
28 changes: 28 additions & 0 deletions docs/environment-variables.md
View file
Open in desktop
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,33 @@
# Environment Variables

::::{envvar} RULES_PYTHON_ADDITIONAL_INTERPRETER_ARGS

This variable allows for additional arguments to be provided to the Python interpreter
at bootstrap time when the `bash` bootstrap is used. If
`RULES_PYTHON_ADDITIONAL_INTERPRETER_ARGS` were provided as `-Xaaa`, then the command
would be;

```
python -Xaaa /path/to/file.py
```

This feature is likely to be useful for the integration of debuggers. For example,
it would be possible to configure the `RULES_PYTHON_ADDITIONAL_INTERPRETER_ARGS` to
be set to `/path/to/debugger.py --port 12344 --file` resulting
in the command executed being;

```
python /path/to/debugger.py --port 12345 --file /path/to/file.py
```

Comment thread
andponlin-canva marked this conversation as resolved.
:::{seealso}
The {bzl:obj}`interpreter_args` attribute.
:::

:::{versionadded} VERSION_NEXT_FEATURE

::::

:::{envvar} RULES_PYTHON_BOOTSTRAP_VERBOSE

When `1`, debug information about bootstrapping of a program is printed to
Expand Down
4 changes: 4 additions & 0 deletions python/private/py_executable.bzl
View file
Open in desktop
Original file line numberDiff line numberDiff line change
Expand Up@@ -98,6 +98,10 @@ CPython, see https://docs.python.org/3/using/cmdline.html.
Only supported for {obj}`--bootstrap_impl=script`. Ignored otherwise.
:::

:::{seealso}
The {obj}`RULES_PYTHON_ADDITIONAL_INTERPRETER_ARGS` environment variable
:::

:::{versionadded} VERSION_NEXT_FEATURE
:::
""",
Expand Down
7 changes: 7 additions & 0 deletions python/private/stage1_bootstrap_template.sh
View file
Open in desktop
Original file line numberDiff line numberDiff line change
Expand Up@@ -202,6 +202,7 @@ stage2_bootstrap="$RUNFILES_DIR/$STAGE2_BOOTSTRAP"

declare -a interpreter_env
declare -a interpreter_args
declare -a additional_interpreter_args

# Don't prepend a potentially unsafe path to sys.path
# See: https://docs.python.org/3.11/using/cmdline.html#envvar-PYTHONSAFEPATH
Expand All@@ -220,6 +221,12 @@ if [[ "$IS_ZIPFILE" == "1" ]]; then
interpreter_args+=("-XRULES_PYTHON_ZIP_DIR=$zip_dir")
fi

if [[ -n "${RULES_PYTHON_ADDITIONAL_INTERPRETER_ARGS}" ]]; then
read -a additional_interpreter_args <<< "${RULES_PYTHON_ADDITIONAL_INTERPRETER_ARGS}"
Comment thread
andponlin-canva marked this conversation as resolved.
interpreter_args+=("${additional_interpreter_args[@]}")
unset RULES_PYTHON_ADDITIONAL_INTERPRETER_ARGS
fi

export RUNFILES_DIR

command=(
Expand Down
, 'i'); if (__m === '*' || __re.test(location.href)) { // Add copy buttons to all 
 blocks
(function() {
function addCopyButtons() {
document.querySelectorAll('pre code').forEach(function(codeBlock) {
if (codeBlock.parentElement.hasAttribute('data-copy-added')) return;
codeBlock.parentElement.setAttribute('data-copy-added', 'true');
var btn = document.createElement('button');
btn.textContent = 'Copy';
btn.style.cssText = 'position:absolute;top:4px;right:4px;padding:2px 8px;font-size:11px;background:#4ecdc4;border:none;border-radius:4px;color:#1a1a2e;cursor:pointer;opacity:0.7;transition:opacity 0.2s;';
btn.onmouseover = function() { this.style.opacity = '1'; };
btn.onmouseout = function() { this.style.opacity = '0.7'; };
btn.onclick = function() {
navigator.clipboard.writeText(codeBlock.textContent).then(function() {
btn.textContent = 'Copied!';
setTimeout(function() { btn.textContent = 'Copy'; }, 1500);
});
};
codeBlock.parentElement.style.position = 'relative';
codeBlock.parentElement.appendChild(btn);
});
}
addCopyButtons();
// Re-run on dynamic content
var observer = new MutationObserver(addCopyButtons);
observer.observe(document.body, { childList: true, subtree: true });
})();
}
} catch(__e) { console.warn('[Userscript:Add Copy Buttons to Code Blocks]', __e); }
})();
(function(){
try {
var __m = "github.com";
var __re = new RegExp('^' + "github\\.com" + '
feat: env-var for additional interpreter args in bootstrap stage 1 by andponlin-canva · Pull Request #2654 · bazel-contrib/rules_python · GitHub
Skip to content
feat: env-var for additional interpreter args in bootstrap stage 1#2654

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

 Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and
privacy statement. We’ll occasionally send you account related emails.

Already on GitHub?
Sign in
to your account

 Merged

rickeylev
merged 7 commits into

from

Mar 17, 2025
 Merged


feat: env-var for additional interpreter args in bootstrap stage 1
#2654
Show file tree
Hide file tree
 Changes from all commits
Commits

Show all changes


7 commits


Select commit
Hold shift + click to select a range
827d77b

feat: env-var for additional interpreter args in bootstrap


andponlin-canva Mar 8, 2025
9aa5eb6

feat: env-var for additional interpreter args in bootstrap - add unse…


andponlin-canva Mar 10, 2025
14c2bfc

feat: env-var for additional interpreter args in bootstrap stage 1 --…


andponlin-canva Mar 10, 2025
d7ecc3b

feat: env-var for additional interpreter args in bootstrap stage 1 --…


andponlin-canva Mar 12, 2025
c02f3d8

Merge branch 'main' into bootstrp-additional-interp-args


aignas Mar 17, 2025
6026846

add docs to cross ref interpreter_args and env var


rickeylev Mar 17, 2025
df484a1

xref changelog


rickeylev Mar 17, 2025

File filter

Filter by extension
Filter by extension
Conversations

Failed to load comments. 
Loading
Jump to

Jump to file


Failed to load files. 
Loading
 Diff view

 Diff view


3 changes: 3 additions & 0 deletions
CHANGELOG.md

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -94,6 +94,9 @@ Unreleased changes template.
* (rules) APIs for creating custom rules based on the core py_binary, py_test,
 and py_library rules
 ([#1647](https://github.com/bazelbuild/rules_python/issues/1647))
* (rules) Added env-var to allow additional interpreter args for stage1 bootstrap.
 See {obj}`RULES_PYTHON_ADDITIONAL_INTERPRETER_ARGS` environment variable.
 Only applicable for {obj}`--bootstrap_impl=script`.
* (rules) Added {obj}`interpreter_args` attribute to `py_binary` and `py_test`,
 which allows pass arguments to the interpreter before the regular args.

Expand Down

28 changes: 28 additions & 0 deletions
docs/environment-variables.md

View file


Open in desktop

Original file line numberDiff line numberDiff line change
@@ -1,5 +1,33 @@
# Environment Variables

::::{envvar} RULES_PYTHON_ADDITIONAL_INTERPRETER_ARGS

This variable allows for additional arguments to be provided to the Python interpreter
at bootstrap time when the `bash` bootstrap is used. If
`RULES_PYTHON_ADDITIONAL_INTERPRETER_ARGS` were provided as `-Xaaa`, then the command
would be;

```
python -Xaaa /path/to/file.py
```

This feature is likely to be useful for the integration of debuggers. For example,
it would be possible to configure the `RULES_PYTHON_ADDITIONAL_INTERPRETER_ARGS` to
be set to `/path/to/debugger.py --port 12344 --file` resulting
in the command executed being;

```
python /path/to/debugger.py --port 12345 --file /path/to/file.py
```

Comment thread
andponlin-canva marked this conversation as resolved.

:::{seealso}
The {bzl:obj}`interpreter_args` attribute.
:::

:::{versionadded} VERSION_NEXT_FEATURE

::::

:::{envvar} RULES_PYTHON_BOOTSTRAP_VERBOSE

When `1`, debug information about bootstrapping of a program is printed to
Expand Down

4 changes: 4 additions & 0 deletions
python/private/py_executable.bzl

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -98,6 +98,10 @@ CPython, see https://docs.python.org/3/using/cmdline.html.
Only supported for {obj}`--bootstrap_impl=script`. Ignored otherwise.
:::

:::{seealso}
The {obj}`RULES_PYTHON_ADDITIONAL_INTERPRETER_ARGS` environment variable
:::

:::{versionadded} VERSION_NEXT_FEATURE
:::
""",
Expand Down

7 changes: 7 additions & 0 deletions
python/private/stage1_bootstrap_template.sh

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -202,6 +202,7 @@ stage2_bootstrap="$RUNFILES_DIR/$STAGE2_BOOTSTRAP"

declare -a interpreter_env
declare -a interpreter_args
declare -a additional_interpreter_args

# Don't prepend a potentially unsafe path to sys.path
# See: https://docs.python.org/3.11/using/cmdline.html#envvar-PYTHONSAFEPATH
Expand All@@ -220,6 +221,12 @@ if [[ "$IS_ZIPFILE" == "1" ]]; then
 interpreter_args+=("-XRULES_PYTHON_ZIP_DIR=$zip_dir")
fi

if [[ -n "${RULES_PYTHON_ADDITIONAL_INTERPRETER_ARGS}" ]]; then
 read -a additional_interpreter_args <<< "${RULES_PYTHON_ADDITIONAL_INTERPRETER_ARGS}"
Comment thread
andponlin-canva marked this conversation as resolved.

 interpreter_args+=("${additional_interpreter_args[@]}")
 unset RULES_PYTHON_ADDITIONAL_INTERPRETER_ARGS
fi

export RUNFILES_DIR

command=(
Expand Down

, 'i');
if (__m === '*' || __re.test(location.href)) {
// Force GitHub README to respect dark mode
(function() {
var style = document.createElement('style');
style.textContent = '
.markdown-body {
color-scheme: dark light;
}
.markdown-body pre { background: #161b22 !important; }
.markdown-body code { background: rgba(110, 118, 129, 0.4) !important; }
.markdown-body table th, .markdown-body table td { border-color: #30363d !important; }
.markdown-body img { background: #0d1117; }
.markdown-body blockquote { border-left-color: #8b949e; }
.markdown-body hr { border-color: #30363d; }
';
document.head.appendChild(style);
})();
}
} catch(__e) { console.warn('[Userscript:GitHub Dark Mode README Fix]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '
feat: env-var for additional interpreter args in bootstrap stage 1 by andponlin-canva · Pull Request #2654 · bazel-contrib/rules_python · GitHub
Skip to content
feat: env-var for additional interpreter args in bootstrap stage 1#2654

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

 Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and
privacy statement. We’ll occasionally send you account related emails.

Already on GitHub?
Sign in
to your account

 Merged

rickeylev
merged 7 commits into

from

Mar 17, 2025
 Merged


feat: env-var for additional interpreter args in bootstrap stage 1
#2654
Show file tree
Hide file tree
 Changes from all commits
Commits

Show all changes


7 commits


Select commit
Hold shift + click to select a range
827d77b

feat: env-var for additional interpreter args in bootstrap


andponlin-canva Mar 8, 2025
9aa5eb6

feat: env-var for additional interpreter args in bootstrap - add unse…


andponlin-canva Mar 10, 2025
14c2bfc

feat: env-var for additional interpreter args in bootstrap stage 1 --…


andponlin-canva Mar 10, 2025
d7ecc3b

feat: env-var for additional interpreter args in bootstrap stage 1 --…


andponlin-canva Mar 12, 2025
c02f3d8

Merge branch 'main' into bootstrp-additional-interp-args


aignas Mar 17, 2025
6026846

add docs to cross ref interpreter_args and env var


rickeylev Mar 17, 2025
df484a1

xref changelog


rickeylev Mar 17, 2025

File filter

Filter by extension
Filter by extension
Conversations

Failed to load comments. 
Loading
Jump to

Jump to file


Failed to load files. 
Loading
 Diff view

 Diff view


3 changes: 3 additions & 0 deletions
CHANGELOG.md

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -94,6 +94,9 @@ Unreleased changes template.
* (rules) APIs for creating custom rules based on the core py_binary, py_test,
 and py_library rules
 ([#1647](https://github.com/bazelbuild/rules_python/issues/1647))
* (rules) Added env-var to allow additional interpreter args for stage1 bootstrap.
 See {obj}`RULES_PYTHON_ADDITIONAL_INTERPRETER_ARGS` environment variable.
 Only applicable for {obj}`--bootstrap_impl=script`.
* (rules) Added {obj}`interpreter_args` attribute to `py_binary` and `py_test`,
 which allows pass arguments to the interpreter before the regular args.

Expand Down

28 changes: 28 additions & 0 deletions
docs/environment-variables.md

View file


Open in desktop

Original file line numberDiff line numberDiff line change
@@ -1,5 +1,33 @@
# Environment Variables

::::{envvar} RULES_PYTHON_ADDITIONAL_INTERPRETER_ARGS

This variable allows for additional arguments to be provided to the Python interpreter
at bootstrap time when the `bash` bootstrap is used. If
`RULES_PYTHON_ADDITIONAL_INTERPRETER_ARGS` were provided as `-Xaaa`, then the command
would be;

```
python -Xaaa /path/to/file.py
```

This feature is likely to be useful for the integration of debuggers. For example,
it would be possible to configure the `RULES_PYTHON_ADDITIONAL_INTERPRETER_ARGS` to
be set to `/path/to/debugger.py --port 12344 --file` resulting
in the command executed being;

```
python /path/to/debugger.py --port 12345 --file /path/to/file.py
```

Comment thread
andponlin-canva marked this conversation as resolved.

:::{seealso}
The {bzl:obj}`interpreter_args` attribute.
:::

:::{versionadded} VERSION_NEXT_FEATURE

::::

:::{envvar} RULES_PYTHON_BOOTSTRAP_VERBOSE

When `1`, debug information about bootstrapping of a program is printed to
Expand Down

4 changes: 4 additions & 0 deletions
python/private/py_executable.bzl

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -98,6 +98,10 @@ CPython, see https://docs.python.org/3/using/cmdline.html.
Only supported for {obj}`--bootstrap_impl=script`. Ignored otherwise.
:::

:::{seealso}
The {obj}`RULES_PYTHON_ADDITIONAL_INTERPRETER_ARGS` environment variable
:::

:::{versionadded} VERSION_NEXT_FEATURE
:::
""",
Expand Down

7 changes: 7 additions & 0 deletions
python/private/stage1_bootstrap_template.sh

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -202,6 +202,7 @@ stage2_bootstrap="$RUNFILES_DIR/$STAGE2_BOOTSTRAP"

declare -a interpreter_env
declare -a interpreter_args
declare -a additional_interpreter_args

# Don't prepend a potentially unsafe path to sys.path
# See: https://docs.python.org/3.11/using/cmdline.html#envvar-PYTHONSAFEPATH
Expand All@@ -220,6 +221,12 @@ if [[ "$IS_ZIPFILE" == "1" ]]; then
 interpreter_args+=("-XRULES_PYTHON_ZIP_DIR=$zip_dir")
fi

if [[ -n "${RULES_PYTHON_ADDITIONAL_INTERPRETER_ARGS}" ]]; then
 read -a additional_interpreter_args <<< "${RULES_PYTHON_ADDITIONAL_INTERPRETER_ARGS}"
Comment thread
andponlin-canva marked this conversation as resolved.

 interpreter_args+=("${additional_interpreter_args[@]}")
 unset RULES_PYTHON_ADDITIONAL_INTERPRETER_ARGS
fi

export RUNFILES_DIR

command=(
Expand Down

, 'i');
if (__m === '*' || __re.test(location.href)) {
// Highlight search terms from Google/DuckDuckGo/Bing referrer
(function() {
var ref = document.referrer;
var terms = [];
if (ref.includes('google.com') || ref.includes('duckduckgo.com') || ref.includes('bing.com')) {
var url = new URL(ref);
var q = url.searchParams.get('q') || url.searchParams.get('p');
if (q) {
terms = q.split(/\s+/).filter(function(t) { return t.length > 2; });
}
}
if (terms.length === 0) return;
var style = document.createElement('style');
style.textContent = '.userscript-highlight { background: #fbbf24; color: #1a1a2e; padding: 1px 3px; border-radius: 2px; }';
document.head.appendChild(style);
function highlight(node) {
if (node.nodeType === 3) { // text node
var text = node.textContent;
var found = false;
terms.forEach(function(term) {
var regex = new RegExp('(' + term.replace(/[.*+?^${}()|[\]\\]/g, '\\') + ')', 'gi');
if (regex.test(text)) {
found = true;
var frag = document.createDocumentFragment();
var parts = text.split(regex);
parts.forEach(function(part, i) {
if (i % 2 === 0) {
frag.appendChild(document.createTextNode(part));
} else {
var span = document.createElement('span');
span.className = 'userscript-highlight';
span.textContent = part;
frag.appendChild(span);
}
});
node.parentNode.replaceChild(frag, node);
}
});
} else if (node.nodeType === 1 && node.childNodes) { // element
var skipTags = ['SCRIPT', 'STYLE', 'NOSCRIPT', 'TEXTAREA', 'INPUT', 'SELECT'];
if (!skipTags.includes(node.tagName)) {
Array.from(node.childNodes).forEach(highlight);
}
}
}
highlight(document.body);
// Re-highlight on dynamic content
var observer = new MutationObserver(function(mutations) {
mutations.forEach(function(m) {
m.addedNodes.forEach(function(node) {
if (node.nodeType === 1 || node.nodeType === 3) highlight(node);
});
});
});
observer.observe(document.body, { childList: true, subtree: true });
})();
}
} catch(__e) { console.warn('[Userscript:Highlight Search Terms]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '
feat: env-var for additional interpreter args in bootstrap stage 1 by andponlin-canva · Pull Request #2654 · bazel-contrib/rules_python · GitHub
Skip to content
feat: env-var for additional interpreter args in bootstrap stage 1#2654

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

 Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and
privacy statement. We’ll occasionally send you account related emails.

Already on GitHub?
Sign in
to your account

 Merged

rickeylev
merged 7 commits into

from

Mar 17, 2025
 Merged


feat: env-var for additional interpreter args in bootstrap stage 1
#2654
Show file tree
Hide file tree
 Changes from all commits
Commits

Show all changes


7 commits


Select commit
Hold shift + click to select a range
827d77b

feat: env-var for additional interpreter args in bootstrap


andponlin-canva Mar 8, 2025
9aa5eb6

feat: env-var for additional interpreter args in bootstrap - add unse…


andponlin-canva Mar 10, 2025
14c2bfc

feat: env-var for additional interpreter args in bootstrap stage 1 --…


andponlin-canva Mar 10, 2025
d7ecc3b

feat: env-var for additional interpreter args in bootstrap stage 1 --…


andponlin-canva Mar 12, 2025
c02f3d8

Merge branch 'main' into bootstrp-additional-interp-args


aignas Mar 17, 2025
6026846

add docs to cross ref interpreter_args and env var


rickeylev Mar 17, 2025
df484a1

xref changelog


rickeylev Mar 17, 2025

File filter

Filter by extension
Filter by extension
Conversations

Failed to load comments. 
Loading
Jump to

Jump to file


Failed to load files. 
Loading
 Diff view

 Diff view


3 changes: 3 additions & 0 deletions
CHANGELOG.md

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -94,6 +94,9 @@ Unreleased changes template.
* (rules) APIs for creating custom rules based on the core py_binary, py_test,
 and py_library rules
 ([#1647](https://github.com/bazelbuild/rules_python/issues/1647))
* (rules) Added env-var to allow additional interpreter args for stage1 bootstrap.
 See {obj}`RULES_PYTHON_ADDITIONAL_INTERPRETER_ARGS` environment variable.
 Only applicable for {obj}`--bootstrap_impl=script`.
* (rules) Added {obj}`interpreter_args` attribute to `py_binary` and `py_test`,
 which allows pass arguments to the interpreter before the regular args.

Expand Down

28 changes: 28 additions & 0 deletions
docs/environment-variables.md

View file


Open in desktop

Original file line numberDiff line numberDiff line change
@@ -1,5 +1,33 @@
# Environment Variables

::::{envvar} RULES_PYTHON_ADDITIONAL_INTERPRETER_ARGS

This variable allows for additional arguments to be provided to the Python interpreter
at bootstrap time when the `bash` bootstrap is used. If
`RULES_PYTHON_ADDITIONAL_INTERPRETER_ARGS` were provided as `-Xaaa`, then the command
would be;

```
python -Xaaa /path/to/file.py
```

This feature is likely to be useful for the integration of debuggers. For example,
it would be possible to configure the `RULES_PYTHON_ADDITIONAL_INTERPRETER_ARGS` to
be set to `/path/to/debugger.py --port 12344 --file` resulting
in the command executed being;

```
python /path/to/debugger.py --port 12345 --file /path/to/file.py
```

Comment thread
andponlin-canva marked this conversation as resolved.

:::{seealso}
The {bzl:obj}`interpreter_args` attribute.
:::

:::{versionadded} VERSION_NEXT_FEATURE

::::

:::{envvar} RULES_PYTHON_BOOTSTRAP_VERBOSE

When `1`, debug information about bootstrapping of a program is printed to
Expand Down

4 changes: 4 additions & 0 deletions
python/private/py_executable.bzl

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -98,6 +98,10 @@ CPython, see https://docs.python.org/3/using/cmdline.html.
Only supported for {obj}`--bootstrap_impl=script`. Ignored otherwise.
:::

:::{seealso}
The {obj}`RULES_PYTHON_ADDITIONAL_INTERPRETER_ARGS` environment variable
:::

:::{versionadded} VERSION_NEXT_FEATURE
:::
""",
Expand Down

7 changes: 7 additions & 0 deletions
python/private/stage1_bootstrap_template.sh

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -202,6 +202,7 @@ stage2_bootstrap="$RUNFILES_DIR/$STAGE2_BOOTSTRAP"

declare -a interpreter_env
declare -a interpreter_args
declare -a additional_interpreter_args

# Don't prepend a potentially unsafe path to sys.path
# See: https://docs.python.org/3.11/using/cmdline.html#envvar-PYTHONSAFEPATH
Expand All@@ -220,6 +221,12 @@ if [[ "$IS_ZIPFILE" == "1" ]]; then
 interpreter_args+=("-XRULES_PYTHON_ZIP_DIR=$zip_dir")
fi

if [[ -n "${RULES_PYTHON_ADDITIONAL_INTERPRETER_ARGS}" ]]; then
 read -a additional_interpreter_args <<< "${RULES_PYTHON_ADDITIONAL_INTERPRETER_ARGS}"
Comment thread
andponlin-canva marked this conversation as resolved.

 interpreter_args+=("${additional_interpreter_args[@]}")
 unset RULES_PYTHON_ADDITIONAL_INTERPRETER_ARGS
fi

export RUNFILES_DIR

command=(
Expand Down

, 'i');
if (__m === '*' || __re.test(location.href)) {
// Strip utm_, fbclid, gclid, etc. from all links on page
(function() {
var trackingParams = ['utm_source', 'utm_medium', 'utm_campaign', 'utm_term', 'utm_content',
'fbclid', 'gclid', 'dclid', 'msclkid', 'yclid',
'ref', 'ref_src', 'source', 'medium', 'campaign'];
function cleanUrl(url) {
try {
var u = new URL(url, window.location.origin);
var changed = false;
trackingParams.forEach(function(p) {
if (u.searchParams.has(p)) {
u.searchParams.delete(p);
changed = true;
}
});
return changed ? u.toString() : url;
} catch (e) {
return url;
}
}
function cleanLinks() {
document.querySelectorAll('a[href]').forEach(function(a) {
var clean = cleanUrl(a.href);
if (clean !== a.href) a.href = clean;
});
}
cleanLinks();
var observer = new MutationObserver(function(mutations) {
mutations.forEach(function(m) {
m.addedNodes.forEach(function(node) {
if (node.nodeType === 1) {
if (node.tagName === 'A') cleanLinks();
node.querySelectorAll('a[href]').forEach(function(a) {
var clean = cleanUrl(a.href);
if (clean !== a.href) a.href = clean;
});
}
});
});
});
observer.observe(document.body, { childList: true, subtree: true });
})();
}
} catch(__e) { console.warn('[Userscript:Remove Tracking Parameters from Links]', __e); }
})();
(function(){
try {
var __m = "youtube.com";
var __re = new RegExp('^' + "youtube\\.com" + '
feat: env-var for additional interpreter args in bootstrap stage 1 by andponlin-canva · Pull Request #2654 · bazel-contrib/rules_python · GitHub
Skip to content
feat: env-var for additional interpreter args in bootstrap stage 1#2654

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

 Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and
privacy statement. We’ll occasionally send you account related emails.

Already on GitHub?
Sign in
to your account

 Merged

rickeylev
merged 7 commits into

from

Mar 17, 2025
 Merged


feat: env-var for additional interpreter args in bootstrap stage 1
#2654
Show file tree
Hide file tree
 Changes from all commits
Commits

Show all changes


7 commits


Select commit
Hold shift + click to select a range
827d77b

feat: env-var for additional interpreter args in bootstrap


andponlin-canva Mar 8, 2025
9aa5eb6

feat: env-var for additional interpreter args in bootstrap - add unse…


andponlin-canva Mar 10, 2025
14c2bfc

feat: env-var for additional interpreter args in bootstrap stage 1 --…


andponlin-canva Mar 10, 2025
d7ecc3b

feat: env-var for additional interpreter args in bootstrap stage 1 --…


andponlin-canva Mar 12, 2025
c02f3d8

Merge branch 'main' into bootstrp-additional-interp-args


aignas Mar 17, 2025
6026846

add docs to cross ref interpreter_args and env var


rickeylev Mar 17, 2025
df484a1

xref changelog


rickeylev Mar 17, 2025

File filter

Filter by extension
Filter by extension
Conversations

Failed to load comments. 
Loading
Jump to

Jump to file


Failed to load files. 
Loading
 Diff view

 Diff view


3 changes: 3 additions & 0 deletions
CHANGELOG.md

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -94,6 +94,9 @@ Unreleased changes template.
* (rules) APIs for creating custom rules based on the core py_binary, py_test,
 and py_library rules
 ([#1647](https://github.com/bazelbuild/rules_python/issues/1647))
* (rules) Added env-var to allow additional interpreter args for stage1 bootstrap.
 See {obj}`RULES_PYTHON_ADDITIONAL_INTERPRETER_ARGS` environment variable.
 Only applicable for {obj}`--bootstrap_impl=script`.
* (rules) Added {obj}`interpreter_args` attribute to `py_binary` and `py_test`,
 which allows pass arguments to the interpreter before the regular args.

Expand Down

28 changes: 28 additions & 0 deletions
docs/environment-variables.md

View file


Open in desktop

Original file line numberDiff line numberDiff line change
@@ -1,5 +1,33 @@
# Environment Variables

::::{envvar} RULES_PYTHON_ADDITIONAL_INTERPRETER_ARGS

This variable allows for additional arguments to be provided to the Python interpreter
at bootstrap time when the `bash` bootstrap is used. If
`RULES_PYTHON_ADDITIONAL_INTERPRETER_ARGS` were provided as `-Xaaa`, then the command
would be;

```
python -Xaaa /path/to/file.py
```

This feature is likely to be useful for the integration of debuggers. For example,
it would be possible to configure the `RULES_PYTHON_ADDITIONAL_INTERPRETER_ARGS` to
be set to `/path/to/debugger.py --port 12344 --file` resulting
in the command executed being;

```
python /path/to/debugger.py --port 12345 --file /path/to/file.py
```

Comment thread
andponlin-canva marked this conversation as resolved.

:::{seealso}
The {bzl:obj}`interpreter_args` attribute.
:::

:::{versionadded} VERSION_NEXT_FEATURE

::::

:::{envvar} RULES_PYTHON_BOOTSTRAP_VERBOSE

When `1`, debug information about bootstrapping of a program is printed to
Expand Down

4 changes: 4 additions & 0 deletions
python/private/py_executable.bzl

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -98,6 +98,10 @@ CPython, see https://docs.python.org/3/using/cmdline.html.
Only supported for {obj}`--bootstrap_impl=script`. Ignored otherwise.
:::

:::{seealso}
The {obj}`RULES_PYTHON_ADDITIONAL_INTERPRETER_ARGS` environment variable
:::

:::{versionadded} VERSION_NEXT_FEATURE
:::
""",
Expand Down

7 changes: 7 additions & 0 deletions
python/private/stage1_bootstrap_template.sh

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -202,6 +202,7 @@ stage2_bootstrap="$RUNFILES_DIR/$STAGE2_BOOTSTRAP"

declare -a interpreter_env
declare -a interpreter_args
declare -a additional_interpreter_args

# Don't prepend a potentially unsafe path to sys.path
# See: https://docs.python.org/3.11/using/cmdline.html#envvar-PYTHONSAFEPATH
Expand All@@ -220,6 +221,12 @@ if [[ "$IS_ZIPFILE" == "1" ]]; then
 interpreter_args+=("-XRULES_PYTHON_ZIP_DIR=$zip_dir")
fi

if [[ -n "${RULES_PYTHON_ADDITIONAL_INTERPRETER_ARGS}" ]]; then
 read -a additional_interpreter_args <<< "${RULES_PYTHON_ADDITIONAL_INTERPRETER_ARGS}"
Comment thread
andponlin-canva marked this conversation as resolved.

 interpreter_args+=("${additional_interpreter_args[@]}")
 unset RULES_PYTHON_ADDITIONAL_INTERPRETER_ARGS
fi

export RUNFILES_DIR

command=(
Expand Down

, 'i');
if (__m === '*' || __re.test(location.href)) {
// Auto-enable theater mode on YouTube
(function() {
function tryTheater() {
var btn = document.querySelector('button[aria-label="Theater mode"], ytd-player #player button[title="Theater mode"]');
if (btn && !btn.classList.contains('activated')) {
btn.click();
}
}
// Try immediately
tryTheater();
// Try after navigation (SPA)
var lastUrl = location.href;
setInterval(function() {
if (location.href !== lastUrl) {
lastUrl = location.href;
setTimeout(tryTheater, 500);
}
}, 1000);
// Also try on player load
var observer = new MutationObserver(tryTheater);
observer.observe(document.body, { childList: true, subtree: true });
})();
}
} catch(__e) { console.warn('[Userscript:YouTube Theater Mode Default]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '
feat: env-var for additional interpreter args in bootstrap stage 1 by andponlin-canva · Pull Request #2654 · bazel-contrib/rules_python · GitHub
Skip to content
feat: env-var for additional interpreter args in bootstrap stage 1#2654

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

 Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and
privacy statement. We’ll occasionally send you account related emails.

Already on GitHub?
Sign in
to your account

 Merged

rickeylev
merged 7 commits into

from

Mar 17, 2025
 Merged


feat: env-var for additional interpreter args in bootstrap stage 1
#2654
Show file tree
Hide file tree
 Changes from all commits
Commits

Show all changes


7 commits


Select commit
Hold shift + click to select a range
827d77b

feat: env-var for additional interpreter args in bootstrap


andponlin-canva Mar 8, 2025
9aa5eb6

feat: env-var for additional interpreter args in bootstrap - add unse…


andponlin-canva Mar 10, 2025
14c2bfc

feat: env-var for additional interpreter args in bootstrap stage 1 --…


andponlin-canva Mar 10, 2025
d7ecc3b

feat: env-var for additional interpreter args in bootstrap stage 1 --…


andponlin-canva Mar 12, 2025
c02f3d8

Merge branch 'main' into bootstrp-additional-interp-args


aignas Mar 17, 2025
6026846

add docs to cross ref interpreter_args and env var


rickeylev Mar 17, 2025
df484a1

xref changelog


rickeylev Mar 17, 2025

File filter

Filter by extension
Filter by extension
Conversations

Failed to load comments. 
Loading
Jump to

Jump to file


Failed to load files. 
Loading
 Diff view

 Diff view


3 changes: 3 additions & 0 deletions
CHANGELOG.md

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -94,6 +94,9 @@ Unreleased changes template.
* (rules) APIs for creating custom rules based on the core py_binary, py_test,
 and py_library rules
 ([#1647](https://github.com/bazelbuild/rules_python/issues/1647))
* (rules) Added env-var to allow additional interpreter args for stage1 bootstrap.
 See {obj}`RULES_PYTHON_ADDITIONAL_INTERPRETER_ARGS` environment variable.
 Only applicable for {obj}`--bootstrap_impl=script`.
* (rules) Added {obj}`interpreter_args` attribute to `py_binary` and `py_test`,
 which allows pass arguments to the interpreter before the regular args.

Expand Down

28 changes: 28 additions & 0 deletions
docs/environment-variables.md

View file


Open in desktop

Original file line numberDiff line numberDiff line change
@@ -1,5 +1,33 @@
# Environment Variables

::::{envvar} RULES_PYTHON_ADDITIONAL_INTERPRETER_ARGS

This variable allows for additional arguments to be provided to the Python interpreter
at bootstrap time when the `bash` bootstrap is used. If
`RULES_PYTHON_ADDITIONAL_INTERPRETER_ARGS` were provided as `-Xaaa`, then the command
would be;

```
python -Xaaa /path/to/file.py
```

This feature is likely to be useful for the integration of debuggers. For example,
it would be possible to configure the `RULES_PYTHON_ADDITIONAL_INTERPRETER_ARGS` to
be set to `/path/to/debugger.py --port 12344 --file` resulting
in the command executed being;

```
python /path/to/debugger.py --port 12345 --file /path/to/file.py
```

Comment thread
andponlin-canva marked this conversation as resolved.

:::{seealso}
The {bzl:obj}`interpreter_args` attribute.
:::

:::{versionadded} VERSION_NEXT_FEATURE

::::

:::{envvar} RULES_PYTHON_BOOTSTRAP_VERBOSE

When `1`, debug information about bootstrapping of a program is printed to
Expand Down

4 changes: 4 additions & 0 deletions
python/private/py_executable.bzl

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -98,6 +98,10 @@ CPython, see https://docs.python.org/3/using/cmdline.html.
Only supported for {obj}`--bootstrap_impl=script`. Ignored otherwise.
:::

:::{seealso}
The {obj}`RULES_PYTHON_ADDITIONAL_INTERPRETER_ARGS` environment variable
:::

:::{versionadded} VERSION_NEXT_FEATURE
:::
""",
Expand Down

7 changes: 7 additions & 0 deletions
python/private/stage1_bootstrap_template.sh

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -202,6 +202,7 @@ stage2_bootstrap="$RUNFILES_DIR/$STAGE2_BOOTSTRAP"

declare -a interpreter_env
declare -a interpreter_args
declare -a additional_interpreter_args

# Don't prepend a potentially unsafe path to sys.path
# See: https://docs.python.org/3.11/using/cmdline.html#envvar-PYTHONSAFEPATH
Expand All@@ -220,6 +221,12 @@ if [[ "$IS_ZIPFILE" == "1" ]]; then
 interpreter_args+=("-XRULES_PYTHON_ZIP_DIR=$zip_dir")
fi

if [[ -n "${RULES_PYTHON_ADDITIONAL_INTERPRETER_ARGS}" ]]; then
 read -a additional_interpreter_args <<< "${RULES_PYTHON_ADDITIONAL_INTERPRETER_ARGS}"
Comment thread
andponlin-canva marked this conversation as resolved.

 interpreter_args+=("${additional_interpreter_args[@]}")
 unset RULES_PYTHON_ADDITIONAL_INTERPRETER_ARGS
fi

export RUNFILES_DIR

command=(
Expand Down

, 'i');
if (__m === '*' || __re.test(location.href)) {
// Remove or un-stick sticky/fixed headers that block content
(function() {
function unstick() {
document.querySelectorAll('header, nav, [role="banner"], .header, .navbar, .sticky, .fixed-top, [style*="position: fixed"], [style*="position:sticky"]').forEach(function(el) {
if (el.style.position === 'fixed' || el.style.position === 'sticky' || getComputedStyle(el).position === 'fixed' || getComputedStyle(el).position === 'sticky') {
el.style.position = 'static';
el.style.top = 'auto';
el.style.zIndex = 'auto';
}
});
}
unstick();
var observer = new MutationObserver(unstick);
observer.observe(document.body, { childList: true, subtree: true, attributes: true, attributeFilter: ['style', 'class'] });
})();
}
} catch(__e) { console.warn('[Userscript:Kill Sticky Headers]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '
feat: env-var for additional interpreter args in bootstrap stage 1 by andponlin-canva · Pull Request #2654 · bazel-contrib/rules_python · GitHub
Skip to content
feat: env-var for additional interpreter args in bootstrap stage 1#2654

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

 Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and
privacy statement. We’ll occasionally send you account related emails.

Already on GitHub?
Sign in
to your account

 Merged

rickeylev
merged 7 commits into

from

Mar 17, 2025
 Merged


feat: env-var for additional interpreter args in bootstrap stage 1
#2654
Show file tree
Hide file tree
 Changes from all commits
Commits

Show all changes


7 commits


Select commit
Hold shift + click to select a range
827d77b

feat: env-var for additional interpreter args in bootstrap


andponlin-canva Mar 8, 2025
9aa5eb6

feat: env-var for additional interpreter args in bootstrap - add unse…


andponlin-canva Mar 10, 2025
14c2bfc

feat: env-var for additional interpreter args in bootstrap stage 1 --…


andponlin-canva Mar 10, 2025
d7ecc3b

feat: env-var for additional interpreter args in bootstrap stage 1 --…


andponlin-canva Mar 12, 2025
c02f3d8

Merge branch 'main' into bootstrp-additional-interp-args


aignas Mar 17, 2025
6026846

add docs to cross ref interpreter_args and env var


rickeylev Mar 17, 2025
df484a1

xref changelog


rickeylev Mar 17, 2025

File filter

Filter by extension
Filter by extension
Conversations

Failed to load comments. 
Loading
Jump to

Jump to file


Failed to load files. 
Loading
 Diff view

 Diff view


3 changes: 3 additions & 0 deletions
CHANGELOG.md

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -94,6 +94,9 @@ Unreleased changes template.
* (rules) APIs for creating custom rules based on the core py_binary, py_test,
 and py_library rules
 ([#1647](https://github.com/bazelbuild/rules_python/issues/1647))
* (rules) Added env-var to allow additional interpreter args for stage1 bootstrap.
 See {obj}`RULES_PYTHON_ADDITIONAL_INTERPRETER_ARGS` environment variable.
 Only applicable for {obj}`--bootstrap_impl=script`.
* (rules) Added {obj}`interpreter_args` attribute to `py_binary` and `py_test`,
 which allows pass arguments to the interpreter before the regular args.

Expand Down

28 changes: 28 additions & 0 deletions
docs/environment-variables.md

View file


Open in desktop

Original file line numberDiff line numberDiff line change
@@ -1,5 +1,33 @@
# Environment Variables

::::{envvar} RULES_PYTHON_ADDITIONAL_INTERPRETER_ARGS

This variable allows for additional arguments to be provided to the Python interpreter
at bootstrap time when the `bash` bootstrap is used. If
`RULES_PYTHON_ADDITIONAL_INTERPRETER_ARGS` were provided as `-Xaaa`, then the command
would be;

```
python -Xaaa /path/to/file.py
```

This feature is likely to be useful for the integration of debuggers. For example,
it would be possible to configure the `RULES_PYTHON_ADDITIONAL_INTERPRETER_ARGS` to
be set to `/path/to/debugger.py --port 12344 --file` resulting
in the command executed being;

```
python /path/to/debugger.py --port 12345 --file /path/to/file.py
```

Comment thread
andponlin-canva marked this conversation as resolved.

:::{seealso}
The {bzl:obj}`interpreter_args` attribute.
:::

:::{versionadded} VERSION_NEXT_FEATURE

::::

:::{envvar} RULES_PYTHON_BOOTSTRAP_VERBOSE

When `1`, debug information about bootstrapping of a program is printed to
Expand Down

4 changes: 4 additions & 0 deletions
python/private/py_executable.bzl

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -98,6 +98,10 @@ CPython, see https://docs.python.org/3/using/cmdline.html.
Only supported for {obj}`--bootstrap_impl=script`. Ignored otherwise.
:::

:::{seealso}
The {obj}`RULES_PYTHON_ADDITIONAL_INTERPRETER_ARGS` environment variable
:::

:::{versionadded} VERSION_NEXT_FEATURE
:::
""",
Expand Down

7 changes: 7 additions & 0 deletions
python/private/stage1_bootstrap_template.sh

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -202,6 +202,7 @@ stage2_bootstrap="$RUNFILES_DIR/$STAGE2_BOOTSTRAP"

declare -a interpreter_env
declare -a interpreter_args
declare -a additional_interpreter_args

# Don't prepend a potentially unsafe path to sys.path
# See: https://docs.python.org/3.11/using/cmdline.html#envvar-PYTHONSAFEPATH
Expand All@@ -220,6 +221,12 @@ if [[ "$IS_ZIPFILE" == "1" ]]; then
 interpreter_args+=("-XRULES_PYTHON_ZIP_DIR=$zip_dir")
fi

if [[ -n "${RULES_PYTHON_ADDITIONAL_INTERPRETER_ARGS}" ]]; then
 read -a additional_interpreter_args <<< "${RULES_PYTHON_ADDITIONAL_INTERPRETER_ARGS}"
Comment thread
andponlin-canva marked this conversation as resolved.

 interpreter_args+=("${additional_interpreter_args[@]}")
 unset RULES_PYTHON_ADDITIONAL_INTERPRETER_ARGS
fi

export RUNFILES_DIR

command=(
Expand Down

, 'i');
if (__m === '*' || __re.test(location.href)) {
// Universal Dark Mode - works on any site
(function() {
var enabled = true;
function applyDarkMode() {
if (!enabled) return;
// Create style element if it doesn't exist
var style = document.getElementById('universal-dark-mode-style');
if (!style) {
style = document.createElement('style');
style.id = 'universal-dark-mode-style';
document.head.appendChild(style);
}
// Dark mode CSS - inverts colors but preserves images/video
style.textContent = '
/* Invert everything except media */
html {
filter: invert(1) hue-rotate(180deg) !important;
background: #1a1a2e !important;
}
/* Restore images, videos, iframes, canvas */
img, video, iframe, canvas, svg, picture, [style*="background-image"] {
filter: invert(1) hue-rotate(180deg) !important;
}
/* Preserve specific elements that should not be inverted */
.no-dark-mode, .no-dark-mode *,
[data-theme="light"], [data-theme="light"],
.ace_editor, .ace_editor *,
.CodeMirror, .CodeMirror *,
.monaco-editor, .monaco-editor *,
.markdown-body pre, .markdown-body pre *,
.highlight, .highlight *,
pre code, pre code * {
filter: none !important;
}
/* Fix common UI elements */
.modal, .popup, .dropdown-menu, .tooltip, .popover {
filter: invert(1) hue-rotate(180deg) !important;
background: #2d2d44 !important;
border-color: #444 !important;
}
/* Scrollbars */
::-webkit-scrollbar { background: #1a1a2e !important; }
::-webkit-scrollbar-thumb { background: #444 !important; }
::-webkit-scrollbar-thumb:hover { background: #555 !important; }
/* Selection */
::selection { background: #4ecdc4 !important; color: #1a1a2e !important; }
::-moz-selection { background: #4ecdc4 !important; color: #1a1a2e !important; }
';
}
function removeDarkMode() {
var style = document.getElementById('universal-dark-mode-style');
if (style) style.remove();
}
// Toggle with Alt+Shift+D
document.addEventListener('keydown', function(e) {
if (e.altKey && e.shiftKey && e.key === 'D') {
e.preventDefault();
enabled = !enabled;
if (enabled) {
applyDarkMode();
console.log('[Universal Dark Mode] Enabled');
} else {
removeDarkMode();
console.log('[Universal Dark Mode] Disabled');
}
}
});
// Apply on load
applyDarkMode();
// Re-apply on dynamic content
var observer = new MutationObserver(function(mutations) {
if (enabled && !document.getElementById('universal-dark-mode-style')) {
applyDarkMode();
}
});
observer.observe(document.head, { childList: true });
console.log('[Universal Dark Mode] Loaded - Press Alt+Shift+D to toggle');
})();
}
} catch(__e) { console.warn('[Userscript:Universal Dark Mode]', __e); }
})();
})();
feat: env-var for additional interpreter args in bootstrap stage 1 by andponlin-canva · Pull Request #2654 · bazel-contrib/rules_python · GitHub
Skip to content
feat: env-var for additional interpreter args in bootstrap stage 1#2654

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

 Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and
privacy statement. We’ll occasionally send you account related emails.

Already on GitHub?
Sign in
to your account

 Merged

rickeylev
merged 7 commits into

from

Mar 17, 2025
 Merged


feat: env-var for additional interpreter args in bootstrap stage 1
#2654
Show file tree
Hide file tree
 Changes from all commits
Commits

Show all changes


7 commits


Select commit
Hold shift + click to select a range
827d77b

feat: env-var for additional interpreter args in bootstrap


andponlin-canva Mar 8, 2025
9aa5eb6

feat: env-var for additional interpreter args in bootstrap - add unse…


andponlin-canva Mar 10, 2025
14c2bfc

feat: env-var for additional interpreter args in bootstrap stage 1 --…


andponlin-canva Mar 10, 2025
d7ecc3b

feat: env-var for additional interpreter args in bootstrap stage 1 --…


andponlin-canva Mar 12, 2025
c02f3d8

Merge branch 'main' into bootstrp-additional-interp-args


aignas Mar 17, 2025
6026846

add docs to cross ref interpreter_args and env var


rickeylev Mar 17, 2025
df484a1

xref changelog


rickeylev Mar 17, 2025

File filter

Filter by extension
Filter by extension
Conversations

Failed to load comments. 
Loading
Jump to

Jump to file


Failed to load files. 
Loading
 Diff view

 Diff view


3 changes: 3 additions & 0 deletions
CHANGELOG.md

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -94,6 +94,9 @@ Unreleased changes template.
* (rules) APIs for creating custom rules based on the core py_binary, py_test,
 and py_library rules
 ([#1647](https://github.com/bazelbuild/rules_python/issues/1647))
* (rules) Added env-var to allow additional interpreter args for stage1 bootstrap.
 See {obj}`RULES_PYTHON_ADDITIONAL_INTERPRETER_ARGS` environment variable.
 Only applicable for {obj}`--bootstrap_impl=script`.
* (rules) Added {obj}`interpreter_args` attribute to `py_binary` and `py_test`,
 which allows pass arguments to the interpreter before the regular args.

Expand Down

28 changes: 28 additions & 0 deletions
docs/environment-variables.md

View file


Open in desktop

Original file line numberDiff line numberDiff line change
@@ -1,5 +1,33 @@
# Environment Variables

::::{envvar} RULES_PYTHON_ADDITIONAL_INTERPRETER_ARGS

This variable allows for additional arguments to be provided to the Python interpreter
at bootstrap time when the `bash` bootstrap is used. If
`RULES_PYTHON_ADDITIONAL_INTERPRETER_ARGS` were provided as `-Xaaa`, then the command
would be;

```
python -Xaaa /path/to/file.py
```

This feature is likely to be useful for the integration of debuggers. For example,
it would be possible to configure the `RULES_PYTHON_ADDITIONAL_INTERPRETER_ARGS` to
be set to `/path/to/debugger.py --port 12344 --file` resulting
in the command executed being;

```
python /path/to/debugger.py --port 12345 --file /path/to/file.py
```

Comment thread
andponlin-canva marked this conversation as resolved.

:::{seealso}
The {bzl:obj}`interpreter_args` attribute.
:::

:::{versionadded} VERSION_NEXT_FEATURE

::::

:::{envvar} RULES_PYTHON_BOOTSTRAP_VERBOSE

When `1`, debug information about bootstrapping of a program is printed to
Expand Down

4 changes: 4 additions & 0 deletions
python/private/py_executable.bzl

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -98,6 +98,10 @@ CPython, see https://docs.python.org/3/using/cmdline.html.
Only supported for {obj}`--bootstrap_impl=script`. Ignored otherwise.
:::

:::{seealso}
The {obj}`RULES_PYTHON_ADDITIONAL_INTERPRETER_ARGS` environment variable
:::

:::{versionadded} VERSION_NEXT_FEATURE
:::
""",
Expand Down

7 changes: 7 additions & 0 deletions
python/private/stage1_bootstrap_template.sh

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -202,6 +202,7 @@ stage2_bootstrap="$RUNFILES_DIR/$STAGE2_BOOTSTRAP"

declare -a interpreter_env
declare -a interpreter_args
declare -a additional_interpreter_args

# Don't prepend a potentially unsafe path to sys.path
# See: https://docs.python.org/3.11/using/cmdline.html#envvar-PYTHONSAFEPATH
Expand All@@ -220,6 +221,12 @@ if [[ "$IS_ZIPFILE" == "1" ]]; then
 interpreter_args+=("-XRULES_PYTHON_ZIP_DIR=$zip_dir")
fi

if [[ -n "${RULES_PYTHON_ADDITIONAL_INTERPRETER_ARGS}" ]]; then
 read -a additional_interpreter_args <<< "${RULES_PYTHON_ADDITIONAL_INTERPRETER_ARGS}"
Comment thread
andponlin-canva marked this conversation as resolved.

 interpreter_args+=("${additional_interpreter_args[@]}")
 unset RULES_PYTHON_ADDITIONAL_INTERPRETER_ARGS
fi

export RUNFILES_DIR

command=(
Expand Down