profile
viewpoint
Julio Merino jmmv Google Inc. New York City, NY, USA https://jmmv.dev/

bazelbuild/sandboxfs 241

A virtual file system for sandboxing

jmmv/kyua 116

Testing framework for infrastructure software

jmmv/atf 89

Libraries to write tests in C, C++ and shell

jmmv/lutok 19

Lightweight C++ API for Lua

jmmv/shtk 19

Application toolkit for programmers writing POSIX-compliant shell scripts

jmmv/sourcachefs 12

Persistent, read-only, FUSE-based caching file system

jmmv/pkg_comp 9

Automates the build of pkgsrc binary packages in a sandbox

jmmv/sysupgrade 8

Automates upgrades of NetBSD

jmmv/endbasic 6

EndBASIC - Simple BASIC interpreter with visual features

jmmv/markdown2social 5

Converts simple Markdown documents to Google+ posts

issue commentbazelbuild/bazel

Clashing Options in Apple Bitcode Bundle on macOS

Assigning to @allevato because he has recently been working in this area and probably knows what the best path is here.

cpsauer

comment created time in 2 days

issue commentbazelbuild/bazel

process-wrapper can kill random process group if PID is reused

This "problem" has existed for a really long time in the process-wrapper, but do you have any evidence that this is a real issue? The kernel tries very hard to not reassign PIDs quickly precisely to avoid issues like this from becoming a problem in practice.

cf-natali

comment created time in 2 days

push eventjmmv/jmmv.dev

Julio Merino

commit sha aeebc4729427d2226b5a2f2722940d88fafa7ae3

Publish rebuilt new site

view details

push time in 2 days

push eventjmmv/jmmv.dev

Julio Merino

commit sha 5af50150b177f3a6d59ba74195f049dc3dbb824a

Publish rebuilt new site

view details

push time in 5 days

push eventjmmv/endbasic

Julio Merino

commit sha e75c2efc91ba1856bb8f8bfa6cf9f1b9395b59df

Fix web build after wasm-bindgen 0.2.66 Do this by allowing mutable globals, which is the current recommendation per https://github.com/rustwasm/wasm-pack/issues/886. Also switch the web integration tests to using Chrome instead of Firefox, as the latter has started failing with an "Invalid Content-type" error that I have not yet been able to tell where it comes from.

view details

push time in 8 days

PR opened jmmv/endbasic

Fix build after wasm-bindgen 0.2.66

Do this by allowing mutable globals, which is the current recommendation per https://github.com/rustwasm/wasm-pack/issues/886.

+5 -0

0 comment

1 changed file

pr created time in 8 days

create barnchjmmv/endbasic

branch : fixes

created branch time in 8 days

push eventjmmv/jmmv.dev

Julio Merino

commit sha 750356efc9f3b92c073e0252b95e2ac593ce891f

Publish rebuilt new site

view details

push time in 11 days

push eventjmmv/jmmv.dev

Julio Merino

commit sha a9afd845f83b87b149fad59fd5e331ba9087c359

Publish rebuilt new site

view details

push time in 11 days

issue commentbazelbuild/bazel

Duplicate debug print statements get filtered from console output

This feature (message deduplication) has caused confusion in the past to the point where some callers implemented workarounds (as explained in this bug) and we had to disable it a year ago within Google (and also in Bazel). Looking at Google-internal usage, nobody seems to be using this any more.

This feature comes with significant complexity and attempts to parse stdout/stderr as text, which arguably we should not be doing because some messages may be binary.

Therefore, I propose that we simply delete --experimental_ui_deduplicate.

brandjon

comment created time in 17 days

push eventjmmv/endbasic

Deployment Bot (from Travis CI)

commit sha 0a9d8d1292eb102cd7a0c18f2a881647cdda13c3

Deploy jmmv/endbasic to github.com/jmmv/endbasic.git:gh-pages

view details

push time in 19 days

push eventjmmv/endbasic

Deployment Bot (from Travis CI)

commit sha 37cbe4e98bd14f397cdf26e2c4bc3ebd8eed32bc

Deploy jmmv/endbasic to github.com/jmmv/endbasic.git:gh-pages

view details

push time in 20 days

delete branch jmmv/sandboxfs

delete branch : fixes

delete time in 20 days

push eventbazelbuild/sandboxfs

Julio Merino

commit sha cdd7315a1b520e1d4207c38351956e1480e408d7

Rename identity_conversion to useless_conversion This check's name has changed in Clippy so adjust it accordingly to avoid inaccurate warnings that break the build.

view details

Julio Merino

commit sha b1ea167b4df2e06a3aa5b222c6edfce5e393da39

Look for unit tests in target/debug/deps I do not know when cargo started putting the unit tests under target/debug/deps instead of target/deps (where we were looking for before) but that seems to be how things work now.

view details

push time in 20 days

PR merged bazelbuild/sandboxfs

Fixes
+10 -7

0 comment

2 changed files

jmmv

pr closed time in 20 days

push eventjmmv/sandboxfs

Julio Merino

commit sha a0a313de21a31222d3f29153d61f5cfefae5328f

Look for unit tests in target/debug/deps I do not know when cargo started putting the unit tests under target/debug/deps instead of target/deps (where we were looking for before) but that seems to be how things work now.

view details

push time in 20 days

delete branch jmmv/sandboxfs

delete branch : clippy

delete time in 20 days

PR opened bazelbuild/sandboxfs

Fixes
+7 -7

0 comment

2 changed files

pr created time in 20 days

create barnchjmmv/sandboxfs

branch : fixes

created branch time in 20 days

push eventjmmv/endbasic

Deployment Bot (from Travis CI)

commit sha 08ef591d8e695fb25b486b7413cd0dc6a7c941e6

Deploy jmmv/endbasic to github.com/jmmv/endbasic.git:gh-pages

view details

push time in 21 days

create barnchjmmv/sandboxfs

branch : clippy

created branch time in 21 days

push eventjmmv/endbasic

Deployment Bot (from Travis CI)

commit sha 3c94b8bee9ba4ac2e2e03c2fd8f28857f1436056

Deploy jmmv/endbasic to github.com/jmmv/endbasic.git:gh-pages

view details

push time in 22 days

push eventjmmv/endbasic

Deployment Bot (from Travis CI)

commit sha 77e71966f347add1b58ff73be45b247b0d254d36

Deploy jmmv/endbasic to github.com/jmmv/endbasic.git:gh-pages

view details

push time in 22 days

PR merged jmmv/endbasic

Detect overflows in arithmetic operations
+54 -6

0 comment

2 changed files

jmmv

pr closed time in 22 days

delete branch jmmv/endbasic

delete branch : overflow

delete time in 22 days

push eventjmmv/endbasic

Julio Merino

commit sha a909818c2219b92485a75db35def8f3097d9b93d

Detect overflows in arithmetic operations

view details

push time in 22 days

push eventjmmv/endbasic

Deployment Bot (from Travis CI)

commit sha 0103b2fdb6e686b5597d8d9d713c57efd063a975

Deploy jmmv/endbasic to github.com/jmmv/endbasic.git:gh-pages

view details

push time in 22 days

push eventjmmv/endbasic

Julio Merino

commit sha cda6ce26e443eaff55dc2084e88800b529b6c264

Detect overflows in arithmetic operations

view details

push time in 22 days

push eventjmmv/endbasic

dependabot[bot]

commit sha 2b2dca17f3a5d5fe36ba5184b7c41a5c962ca80f

Bump lodash from 4.17.15 to 4.17.19 in /web/www Bumps [lodash](https://github.com/lodash/lodash) from 4.17.15 to 4.17.19. - [Release notes](https://github.com/lodash/lodash/releases) - [Commits](https://github.com/lodash/lodash/compare/4.17.15...4.17.19) Signed-off-by: dependabot[bot] <support@github.com>

view details

dependabot[bot]

commit sha 2e5baf93a8f813f6df54fe627631e39a9ed710ec

Bump websocket-extensions from 0.1.3 to 0.1.4 in /web/www Bumps [websocket-extensions](https://github.com/faye/websocket-extensions-node) from 0.1.3 to 0.1.4. - [Release notes](https://github.com/faye/websocket-extensions-node/releases) - [Changelog](https://github.com/faye/websocket-extensions-node/blob/master/CHANGELOG.md) - [Commits](https://github.com/faye/websocket-extensions-node/compare/0.1.3...0.1.4) Signed-off-by: dependabot[bot] <support@github.com>

view details

Julio Merino

commit sha fbbbd8a5dd045773c1e127032e95ea670cee3304

Merge branch 'master' into overflow

view details

push time in 22 days

push eventjmmv/endbasic

dependabot[bot]

commit sha 2e5baf93a8f813f6df54fe627631e39a9ed710ec

Bump websocket-extensions from 0.1.3 to 0.1.4 in /web/www Bumps [websocket-extensions](https://github.com/faye/websocket-extensions-node) from 0.1.3 to 0.1.4. - [Release notes](https://github.com/faye/websocket-extensions-node/releases) - [Changelog](https://github.com/faye/websocket-extensions-node/blob/master/CHANGELOG.md) - [Commits](https://github.com/faye/websocket-extensions-node/compare/0.1.3...0.1.4) Signed-off-by: dependabot[bot] <support@github.com>

view details

push time in 22 days

PR merged jmmv/endbasic

Bump websocket-extensions from 0.1.3 to 0.1.4 in /web/www dependencies

Bumps websocket-extensions from 0.1.3 to 0.1.4. <details> <summary>Changelog</summary> <p><em>Sourced from <a href="https://github.com/faye/websocket-extensions-node/blob/master/CHANGELOG.md">websocket-extensions's changelog</a>.</em></p> <blockquote> <h3>0.1.4 / 2020-06-02</h3> <ul> <li>Remove a ReDoS vulnerability in the header parser (CVE-2020-7662, reported by Robert McLaughlin)</li> <li>Change license from MIT to Apache 2.0</li> </ul> </blockquote> </details> <details> <summary>Commits</summary> <ul> <li><a href="https://github.com/faye/websocket-extensions-node/commit/8efd0cd6e35faf9bb9cb08759be1e27082177d43"><code>8efd0cd</code></a> Bump version to 0.1.4</li> <li><a href="https://github.com/faye/websocket-extensions-node/commit/3dad4ad44a8c5f74d4f8f4efd3f9d6e0b5df3051"><code>3dad4ad</code></a> Remove ReDoS vulnerability in the Sec-WebSocket-Extensions header parser</li> <li><a href="https://github.com/faye/websocket-extensions-node/commit/4a76c75efb1c5d6a2f60550e9501757458d19533"><code>4a76c75</code></a> Add Node versions 13 and 14 on Travis</li> <li><a href="https://github.com/faye/websocket-extensions-node/commit/44a677a9c0631daed0b0f4a4b68c095b624183b8"><code>44a677a</code></a> Formatting change: {...} should have spaces inside the braces</li> <li><a href="https://github.com/faye/websocket-extensions-node/commit/f6c50aba0c20ff45b0f87cea33babec1217ec3f5"><code>f6c50ab</code></a> Let npm reformat package.json</li> <li><a href="https://github.com/faye/websocket-extensions-node/commit/2d211f3705d52d9efb4f01daf5a253adf828592e"><code>2d211f3</code></a> Change markdown formatting of docs.</li> <li><a href="https://github.com/faye/websocket-extensions-node/commit/0b620834cc1e1f2eace1d55ab17f71d90d88271d"><code>0b62083</code></a> Update Travis target versions.</li> <li><a href="https://github.com/faye/websocket-extensions-node/commit/729a4653073fa8dd020561113513bfa2e2119415"><code>729a465</code></a> Switch license to Apache 2.0.</li> <li>See full diff in <a href="https://github.com/faye/websocket-extensions-node/compare/0.1.3...0.1.4">compare view</a></li> </ul> </details> <br />

Dependabot compatibility score

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


<details> <summary>Dependabot commands and options</summary> <br />

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)
  • @dependabot use these labels will set the current labels as the default for future PRs for this repo and language
  • @dependabot use these reviewers will set the current reviewers as the default for future PRs for this repo and language
  • @dependabot use these assignees will set the current assignees as the default for future PRs for this repo and language
  • @dependabot use this milestone will set the current milestone as the default for future PRs for this repo and language

You can disable automated security fix PRs for this repo from the Security Alerts page.

</details>

+3 -3

0 comment

1 changed file

dependabot[bot]

pr closed time in 22 days

push eventjmmv/endbasic

Deployment Bot (from Travis CI)

commit sha 0cfb632b5e91351030601cc479d11e3a710edcf7

Deploy jmmv/endbasic to github.com/jmmv/endbasic.git:gh-pages

view details

push time in 22 days

PR opened jmmv/endbasic

Detect overflows in arithmetic operations
+54 -6

0 comment

2 changed files

pr created time in 22 days

push eventjmmv/endbasic

dependabot[bot]

commit sha 2b2dca17f3a5d5fe36ba5184b7c41a5c962ca80f

Bump lodash from 4.17.15 to 4.17.19 in /web/www Bumps [lodash](https://github.com/lodash/lodash) from 4.17.15 to 4.17.19. - [Release notes](https://github.com/lodash/lodash/releases) - [Commits](https://github.com/lodash/lodash/compare/4.17.15...4.17.19) Signed-off-by: dependabot[bot] <support@github.com>

view details

Julio Merino

commit sha 108c31c406d552dce6e5971e4a751d2fe9f92536

Merge branch 'master' into dependabot/npm_and_yarn/web/www/websocket-extensions-0.1.4

view details

push time in 22 days

push eventjmmv/endbasic

dependabot[bot]

commit sha 2b2dca17f3a5d5fe36ba5184b7c41a5c962ca80f

Bump lodash from 4.17.15 to 4.17.19 in /web/www Bumps [lodash](https://github.com/lodash/lodash) from 4.17.15 to 4.17.19. - [Release notes](https://github.com/lodash/lodash/releases) - [Commits](https://github.com/lodash/lodash/compare/4.17.15...4.17.19) Signed-off-by: dependabot[bot] <support@github.com>

view details

push time in 22 days

PR merged jmmv/endbasic

Bump lodash from 4.17.15 to 4.17.19 in /web/www dependencies

Bumps lodash from 4.17.15 to 4.17.19. <details> <summary>Release notes</summary> <p><em>Sourced from <a href="https://github.com/lodash/lodash/releases">lodash's releases</a>.</em></p> <blockquote> <h2>4.17.16</h2> </blockquote> </details> <details> <summary>Commits</summary> <ul> <li><a href="https://github.com/lodash/lodash/commit/d7fbc52ee0466a6d248f047b5d5c3e6d1e099056"><code>d7fbc52</code></a> Bump to v4.17.19</li> <li><a href="https://github.com/lodash/lodash/commit/2e1c0f22f425e9c013815b2cd7c2ebd51f49a8d6"><code>2e1c0f2</code></a> Add npm-package</li> <li><a href="https://github.com/lodash/lodash/commit/1b6c282299f4e0271f932b466c67f0f822aa308e"><code>1b6c282</code></a> Bump to v4.17.18</li> <li><a href="https://github.com/lodash/lodash/commit/a370ac81408de2da77a82b3c4b61a01a3b9c2fac"><code>a370ac8</code></a> Bump to v4.17.17</li> <li><a href="https://github.com/lodash/lodash/commit/1144918f3578a84fcc4986da9b806e63a6175cbb"><code>1144918</code></a> Rebuild lodash and docs</li> <li><a href="https://github.com/lodash/lodash/commit/3a3b0fd339c2109563f7e8167dc95265ed82ef3e"><code>3a3b0fd</code></a> Bump to v4.17.16</li> <li><a href="https://github.com/lodash/lodash/commit/c84fe82760fb2d3e03a63379b297a1cc1a2fce12"><code>c84fe82</code></a> fix(zipObjectDeep): prototype pollution (<a href="https://github-redirect.dependabot.com/lodash/lodash/issues/4759">#4759</a>)</li> <li><a href="https://github.com/lodash/lodash/commit/e7b28ea6cb17b4ca021e7c9d66218c8c89782f32"><code>e7b28ea</code></a> Sanitize sourceURL so it cannot affect evaled code (<a href="https://github-redirect.dependabot.com/lodash/lodash/issues/4518">#4518</a>)</li> <li><a href="https://github.com/lodash/lodash/commit/0cec225778d4ac26c2bac95031ecc92a94f08bbb"><code>0cec225</code></a> Fix lodash.isEqual for circular references (<a href="https://github-redirect.dependabot.com/lodash/lodash/issues/4320">#4320</a>) (<a href="https://github-redirect.dependabot.com/lodash/lodash/issues/4515">#4515</a>)</li> <li><a href="https://github.com/lodash/lodash/commit/94c3a8133cb4fcdb50db72b4fd14dd884b195cd5"><code>94c3a81</code></a> Document matches* shorthands for over* methods (<a href="https://github-redirect.dependabot.com/lodash/lodash/issues/4510">#4510</a>) (<a href="https://github-redirect.dependabot.com/lodash/lodash/issues/4514">#4514</a>)</li> <li>Additional commits viewable in <a href="https://github.com/lodash/lodash/compare/4.17.15...4.17.19">compare view</a></li> </ul> </details> <details> <summary>Maintainer changes</summary> <p>This version was pushed to npm by <a href="https://www.npmjs.com/~mathias">mathias</a>, a new releaser for lodash since your current version.</p> </details> <br />

Dependabot compatibility score

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


<details> <summary>Dependabot commands and options</summary> <br />

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)
  • @dependabot use these labels will set the current labels as the default for future PRs for this repo and language
  • @dependabot use these reviewers will set the current reviewers as the default for future PRs for this repo and language
  • @dependabot use these assignees will set the current assignees as the default for future PRs for this repo and language
  • @dependabot use this milestone will set the current milestone as the default for future PRs for this repo and language

You can disable automated security fix PRs for this repo from the Security Alerts page.

</details>

+3 -3

0 comment

1 changed file

dependabot[bot]

pr closed time in 22 days

push eventjmmv/endbasic

Julio Merino

commit sha 68efdbc12fcd22b04419c6b11d809e75fca5529f

Address some extra redundant_field_names warnings

view details

Julio Merino

commit sha 2af8fd03e919876dcd3f801375f01e91b6ebd8bc

Merge branch 'master' into dependabot/npm_and_yarn/web/www/websocket-extensions-0.1.4

view details

push time in 22 days

create barnchjmmv/endbasic

branch : overflow

created branch time in 22 days

push eventjmmv/endbasic

Deployment Bot (from Travis CI)

commit sha 3fea71f7471134a673ab66e5bf66dc5d3ca2eabb

Deploy jmmv/endbasic to github.com/jmmv/endbasic.git:gh-pages

view details

push time in 23 days

Pull request review commentbazelbuild/bazel

Close VanillaJavaBuilder before writing output, in persistent worker mode

 private static int runPersistentWorker() {         if (request == null) {           break;         }+        WorkResponse response;         try (VanillaJavaBuilder builder = new VanillaJavaBuilder()) {           VanillaJavaBuilderResult result = builder.run(request.getArgumentsList());-          WorkResponse response =+          response =

I'm not sure what you mean about the original code. If I click on "Files changed" in this PR, I'm seeing the final state of the change, and I commented on that. But... the GitHub UI always confuses me, so what am I missing?

asuffield

comment created time in 23 days

issue closedbazelbuild/bazel

Pulling images from GCR fails if gcloud auth helper is in use

ATTENTION! Please read and follow:

  • if this is a question about how to build / test / query / deploy using Bazel, or a discussion starter, send it to bazel-discuss@googlegroups.com
  • if this is a bug or feature request, fill the form below as best as you can.

Description of the problem / feature request:

Bazel build/tests requiring external docker images from GCR fails with Operation not permitted on macOS 10.15.5 if docker is configured to use cloud authentication helper.

WARNING: Could not setup log file in <homedir>/.config/gcloud/logs, (IOError: [Errno 1] Operation not permitted: u'<homedir>/.config/gcloud/logs/2020.06.16/08.14.38.216798.log')
ERROR: (gcloud.auth.docker-helper) Unable to create private file [<homedir>/.config/gcloud/credentials.db]: [Errno 1] Operation not permitted: '<homedir>/.config/gcloud/credentials.db'

If --spawn_strategy=standalone is set the everything is working fine.

Adding bazel-real (/usr/local/Cellar/bazel/3.2.0/libexec/bin/bazel-real) with Full disk access (and obviously restarting the daemon) doesn't help.

Feature requests: what underlying problem are you trying to solve with this feature?

Replace this line with your answer.

Bugs: what's the simplest, easiest way to reproduce this bug? Please provide a minimal example if possible.

On macOS Catalina 10.15.5:

  • Install Google Cloud SDK
  • Install docker or create ~/.docker/config.json file
  • Configure Docker to use gcloud helper to authenticate in GCR: docker auth login ; gcloud auth configure-docker

What operating system are you running Bazel on?

macOS 10.15.5

What's the output of bazel info release?

release 3.2.0-homebrew

If bazel info release returns "development version" or "(@non-git)", tell us how you built Bazel.

Replace this line with your answer.

What's the output of git remote get-url origin ; git rev-parse master ; git rev-parse HEAD ?

The issue is not about git.

Have you found anything relevant by searching the web?

Nothing related to macOS access control system.

Any other information, logs, or outputs that you want to share?

Everything is in the description section.

closed time in 23 days

teran

issue commentbazelbuild/bazel

Pulling images from GCR fails if gcloud auth helper is in use

That sounds reasonable but I wondering if bazel have mechanisms allowing to return EPERM error on filesystem access?

I'm not sure what you are asking for there. The gcloud call is already failing with an EPERM, isn't it?

Note that Bazel just runs a subprocess. It has no way to know what that subprocess will do I/O-wise upfront. That's why the sandbox exists: to prevent the subprocess accessing state it's not supposed to. But then, if the subprocess does any of this, you should expect it to fail.

I've just also tested running exactly the same configuration on Linux with gcloud CLI and also doesn't work but with another error (below). So it looks you're right about sandboxing and it's not about macOS itself.

Note that the Linux and macOS sandboxes are not exactly equivalent -- and they cannot be because each OS offers different functionality for sandboxing. But yes, they both catch the problem of the tool trying to write into the user's home directory (with different error messages).

And this is very much intentional. Either whatever rule that runs docker should tell gcloud to put its configuration files elsewhere, or the rule should be marked as no-sandbox to always ignore the sandbox.

teran

comment created time in 23 days

push eventjmmv/endbasic

Deployment Bot (from Travis CI)

commit sha d37d4acff0716ad36144d040b81d997842222675

Deploy jmmv/endbasic to github.com/jmmv/endbasic.git:gh-pages

view details

push time in 24 days

push eventjmmv/endbasic

Deployment Bot (from Travis CI)

commit sha 3753801bf6bd8e5a0736b1ad07c203a719fdd7d2

Deploy jmmv/endbasic to github.com/jmmv/endbasic.git:gh-pages

view details

push time in 25 days

Pull request review commentbazelbuild/bazel

Close VanillaJavaBuilder before writing output, in persistent worker mode

 private static int runPersistentWorker() {         if (request == null) {           break;         }+        WorkResponse response;         try (VanillaJavaBuilder builder = new VanillaJavaBuilder()) {           VanillaJavaBuilderResult result = builder.run(request.getArgumentsList());-          WorkResponse response =+          response =

This seems a bit fragile / confusing to me. Wouldn't something like this also work and make it clearer that the try block is specifically for the VanillaJavaBuilder?

VanillaJavaBuilderResult result;
try (...) {
  result = ...
}
WorkResponse response = ...;
asuffield

comment created time in 25 days

push eventjmmv/endbasic

Deployment Bot (from Travis CI)

commit sha 982a5111de5c896c7429dc52c6720accfd014637

Deploy jmmv/endbasic to github.com/jmmv/endbasic.git:gh-pages

view details

push time in a month

push eventjmmv/endbasic

Deployment Bot (from Travis CI)

commit sha 675a7e14acf959a37fb858aca97fd1991b12ea03

Deploy jmmv/endbasic to github.com/jmmv/endbasic.git:gh-pages

view details

push time in a month

push eventjmmv/endbasic

Deployment Bot (from Travis CI)

commit sha 875bcc795738f7293d7e8922515802dee7eb8e76

Deploy jmmv/endbasic to github.com/jmmv/endbasic.git:gh-pages

view details

push time in a month

push eventjmmv/endbasic

Deployment Bot (from Travis CI)

commit sha 91131d5590ac6b548583c03ec8f1c3742e82fc24

Deploy jmmv/endbasic to github.com/jmmv/endbasic.git:gh-pages

view details

push time in a month

push eventjmmv/jmmv.dev

Julio Merino

commit sha c033cc16b4eb1f7ed1aacb16eb465802b8d27b15

Publish rebuilt new site

view details

push time in a month

push eventjmmv/endbasic

Deployment Bot (from Travis CI)

commit sha 2d5c515d48d9867dee2c440828fd4a1210646f74

Deploy jmmv/endbasic to github.com/jmmv/endbasic.git:gh-pages

view details

push time in a month

push eventjmmv/endbasic

Deployment Bot (from Travis CI)

commit sha 939c602a2f2d84fa36e1d08539c5d0145a554d77

Deploy jmmv/endbasic to github.com/jmmv/endbasic.git:gh-pages

view details

push time in a month

push eventjmmv/endbasic

Deployment Bot (from Travis CI)

commit sha b661ac8adfeadfa02d75c2ab5becfe4b8bd5515c

Deploy jmmv/endbasic to github.com/jmmv/endbasic.git:gh-pages

view details

push time in a month

push eventjmmv/endbasic

Deployment Bot (from Travis CI)

commit sha aa1f4788ec43a75df089a6a4592de66aa90e4741

Deploy jmmv/endbasic to github.com/jmmv/endbasic.git:gh-pages

view details

push time in a month

push eventjmmv/jmmv.dev

Julio Merino

commit sha c3790890cc4b1111a0912329847b304128258d86

Publish rebuilt new site

view details

push time in a month

push eventjmmv/endbasic

Deployment Bot (from Travis CI)

commit sha 72723759466076a784b6f07427ef0a5992186e26

Deploy jmmv/endbasic to github.com/jmmv/endbasic.git:gh-pages

view details

push time in a month

push eventjmmv/endbasic

Deployment Bot (from Travis CI)

commit sha b01941cc8f48eb03e0f477303080ede136cf2496

Deploy jmmv/endbasic to github.com/jmmv/endbasic.git:gh-pages

view details

push time in a month

push eventjmmv/endbasic

Deployment Bot (from Travis CI)

commit sha bcc6c29d766f23b7850f999302f4caca56a06f9d

Deploy jmmv/endbasic to github.com/jmmv/endbasic.git:gh-pages

view details

push time in a month

Pull request review commentbazelbuild/bazel-skylib

Add process wrapper

 bzl_library(     name = "old_sets",     srcs = ["old_sets.bzl"], )++cc_binary(+    name = "process_wrapper",+    srcs = [+        "process_wrapper/process_wrapper.cc",

If that's the code snippet you tried and failed... the : is wrong in the default value. Should be a slash.

jelmansouri

comment created time in a month

Pull request review commentbazelbuild/bazel-skylib

Add process wrapper

 bzl_library(     name = "old_sets",     srcs = ["old_sets.bzl"], )++cc_binary(+    name = "process_wrapper",+    srcs = [+        "process_wrapper/process_wrapper.cc",

@bazel_skylib//lib/process_wrapper:process_wrapper is equivalent to @bazel_skylib//lib/process_wrapper (the default target name is the package's leaf name).

jelmansouri

comment created time in a month

Pull request review commentbazelbuild/bazel-skylib

Add process wrapper

+## process_wrapper++Process wrapper is a helper that allows you, in a platform independent way,+to not depend on run_shell to perform basic operations like capturing+the output or having $pwd used in command line arguments or environment+variables++It is meant to be used in rules implementations like such:++```python+  def _impl(ctx):+    stdout_output = ctx.actions.declare_file(ctx.label.name + ".stdout")+    args = ctx.actions.args()+    args.add("--stdout-file", stdout_output.path)+    args.add("--subst-pwd")+    args.add("--")+    args.add(ctx.executable._compiler.path)+    args.add(ctx.attr.test_config)+    args.add("--current-dir", "<pwd>")+    env = {"CURRENT_DIR": "<pwd>/test_path"}++    ctx.actions.run(+        executable = ctx.executable._process_wrapper,+        inputs = ctx.files.env_files + ctx.files.arg_files,+        outputs = [stdout_output],+        arguments = [args],+        env = env,+        tools = [ctx.executable._process_wrapper_tester],+    )++    return [DefaultInfo(files = depset([stdout_output]))]++  process_wrapper_tester = rule(+      implementation = _impl,+      attrs = {+          "_compiler": attr.label(+              default = "//compiler/label",+              executable = True,+              cfg = "exec",+          ),+          "_process_wrapper": attr.label(+              default = "@bazel_skylib//lib:process_wrapper",+              executable = True,+              allow_single_file = True,+              cfg = "exec",+          ),+      },+  )++```++### Parameters++<table class="params-table">+  <colgroup>+    <col class="col-param" />+    <col class="col-description" />+  </colgroup>+  <tbody>+    <tr id="process_wrapper---">+      <td><code>-- executable_path [args]</code></td>+      <td>+        required.+        <p>+          Everything after -- is used as command line arguments to the child process.+        </p>+        <p>+          executable_path: path to the executable that is going to be launched as a child process.+        </p>+        <p>+          [args]: Arguments to be passed on to the child process.+        </p>+      </td>+    </tr>+    <tr id="process_wrapper-subst-pwd">

Agreed. And even make this not rely on the environment. Make this --subst=var=value so this can apply arbitrary substitutions.

jelmansouri

comment created time in a month

Pull request review commentbazelbuild/bazel-skylib

Add process wrapper

+#include "lib/process_wrapper/system.h"+#include "lib/process_wrapper/utils.h"++// posix headers+#include <fcntl.h>+#include <sys/stat.h>+#include <sys/types.h>+#include <sys/wait.h>+#include <unistd.h>++#include <iostream>+#include <vector>++namespace process_wrapper {++namespace {++class StdoutPipe {+ public:+  static constexpr size_t kReadEndDesc = 0;+  static constexpr size_t kWriteEndDesc = 1;++  ~StdoutPipe() {+    CloseReadEnd();+    CloseWriteEnd();+  }++  int CreateEnds() {+    if (pipe(stdout_pipe_desc_) != 0) {+      std::cerr << "process wrapper error: failed to open the stdout pipes."+                << std::endl;+      return false;+    }+    return true;+  }+  void DupWriteEnd() {+    dup2(stdout_pipe_desc_[kWriteEndDesc], STDOUT_FILENO);+    CloseReadEnd();+    CloseWriteEnd();+  }++  void CloseReadEnd() { Close(kReadEndDesc); }+  void CloseWriteEnd() { Close(kWriteEndDesc); }++  int ReadEndDesc() const { return stdout_pipe_desc_[kReadEndDesc]; }+  int WriteEndDesc() const { return stdout_pipe_desc_[kWriteEndDesc]; }++  bool WriteToFile(const System::StrType& stdout_file) {+    CloseWriteEnd();++    constexpr size_t kBufferSize = 4096;+    char buffer[kBufferSize];+    int stdout_file_desc =+        open(stdout_file.c_str(), O_WRONLY | O_CREAT | O_TRUNC,+             S_IRUSR | S_IWUSR | S_IRGRP | S_IROTH);+    if (stdout_file_desc == -1) {+      std::cerr << "process wrapper error: failed to open redirection file."+                << std::endl;+      return false;+    }+    while (1) {+      ssize_t read_bytes = read(ReadEndDesc(), buffer, kBufferSize);+      if (read_bytes < 0) {+        std::cerr << "process wrapper error: failed child process stdout."+                  << std::endl;+        return false;+      } else if (read_bytes == 0) {+        break;+      }+      ssize_t written_bytes = write(stdout_file_desc, buffer, read_bytes);+      if (written_bytes < 0 || written_bytes != read_bytes) {+        std::cerr << "process wrapper error: failed to write to stdout file."+                  << std::endl;+        return false;+      }+    }++    CloseReadEnd();+    close(stdout_file_desc);+    return true;+  }++ private:+  void Close(size_t idx) {+    if (stdout_pipe_desc_[idx] > 0) {+      close(stdout_pipe_desc_[idx]);+    }+    stdout_pipe_desc_[idx] = -1;+  }+  int stdout_pipe_desc_[2] = {-1};+};++}  // namespace++System::StrType System::GetWorkingDirectory() {+  const size_t kMaxBufferLength = 4096;+  char cwd[kMaxBufferLength];+  if (getcwd(cwd, sizeof(cwd)) == NULL) {+    return System::StrType{};+  }+  return System::StrType{cwd};+}++int System::Exec(const System::StrType& executable,+                 const System::Arguments& arguments,+                 const System::EnvironmentBlock& environment_block,+                 const StrType& stdout_file) {+  StdoutPipe stdout_pipe;+  if (!stdout_file.empty() && !stdout_pipe.CreateEnds()) {+    return -1;+  }++  pid_t child_pid = fork();

Why do we need to fork at all? From what I see in the description of this wrapper, there is no need to do anything upon process termination, so the wrapper can set things up and then give control to the wrapper process, keeping the process tree cleaner (and being much faster e.g. on macOS).

jelmansouri

comment created time in a month

Pull request review commentbazelbuild/bazel-skylib

Add process wrapper

 bzl_library(     name = "old_sets",     srcs = ["old_sets.bzl"], )++cc_binary(+    name = "process_wrapper",+    srcs = [+        "process_wrapper/process_wrapper.cc",

Any reason for not adding a process_wrapper/BUILD file instead?

jelmansouri

comment created time in a month

Pull request review commentbazelbuild/bazel-skylib

Add process wrapper

+#include <fstream>+#include <iostream>+#include <string>++#include "lib/process_wrapper/system.h"+#include "lib/process_wrapper/utils.h"++using CharType = process_wrapper::System::StrType::value_type;++// Simple process wrapper allowing us to not depend on the shell to run a+// process to perform basic operations like capturing the output or having+// the $pwd used in command line arguments or environment variables+int PW_MAIN(int argc, const CharType* argv[], const CharType* envp[]) {+  using namespace process_wrapper;++  System::EnvironmentBlock environment_block;+  // Taking all environment variables from the current process+  // and sending them down to the child process+  for (int i = 0; envp[i] != nullptr; ++i) {+    environment_block.push_back(envp[i]);+  }++  System::StrType exec_path;+  System::StrType touch_file;+  System::StrType stdout_file;+  System::Arguments arguments;+  System::Arguments file_arguments;+  bool subst_pwd = false;++  // Processing current process argument list until -- is encountered+  // everthing after gets sent down to the child process+  for (int i = 1; i < argc; ++i) {+    System::StrType arg = argv[i];+    if (arg == PW_SYS_STR("--subst-pwd")) {+      subst_pwd = true;+    } else {+      // All argument in the else block have a parameter+      if (++i == argc) {+        std::cerr << "process wrapper error: argument \"" << ToUtf8(arg)+                  << "\" missing parameter." << std::endl;+        return -1;+      }+      if (arg == PW_SYS_STR("--env-file")) {+        if (!ReadFileToArray(argv[i], environment_block)) {+          return -1;+        }+      } else if (arg == PW_SYS_STR("--arg-file")) {+        if (!ReadFileToArray(argv[i], file_arguments)) {+          return -1;+        }+      } else if (arg == PW_SYS_STR("--touch-file")) {+        if (!touch_file.empty()) {+          std::cerr << "process wrapper error: \"--touch-file\" appears more "+                       "than once"+                    << std::endl;

Please avoid using std::endl and instead put \n in the strings. std::endl forces a flush of the stream and we don't need this here, and that's already implied by the use of stderr.

jelmansouri

comment created time in a month

push eventjmmv/endbasic

Deployment Bot (from Travis CI)

commit sha e41134eb2da80d4a6f38802bdd5eda9fe83b76e0

Deploy jmmv/endbasic to github.com/jmmv/endbasic.git:gh-pages

view details

push time in a month

pull request commentbazelbuild/bazel-skylib

Add process wrapper

Given that Bazel already has a process-wrapper tool, should this be called something else?

jelmansouri

comment created time in a month

issue closedbazelbuild/bazel

'IllegalStateException` creating a symlink via `ctx.actions`

Description of the problem

Attempting to create a symlink that points to the output of another rule causes an IllegalStateException to be thrown.

Bugs: what's the simplest, easiest way to reproduce this bug? Please provide a minimal example if possible.

I have attached a zip that reliably causes the problem. The .bazelversion is the one that I used for testing, but it's pretty recent.

What operating system are you running Bazel on?

macOS

What's the output of bazel info release?

release 3.2.0rc1

symlink-bug-repo.zip

closed time in a month

shs96c

issue commentbazelbuild/bazel

'IllegalStateException` creating a symlink via `ctx.actions`

Given that @tetromino couldn't reproduce this issue with your same version of Bazel and the same scenario... we'll need more data. Happy to reopen this once we have it.

shs96c

comment created time in a month

issue commentbazelbuild/bazel

'failed: I/O exception during sandboxed execution: null' error when running tests using Bazel and remote caching

We'll need more information to further debug this, as @tetromino found that there are too many places where this error might be coming from. If you can supply such information, we'll reopen this bug. Thanks!

brianoh1979

comment created time in a month

issue closedbazelbuild/bazel

'failed: I/O exception during sandboxed execution: null' error when running tests using Bazel and remote caching

ATTENTION! Please read and follow:

  • if this is a question about how to build / test / query / deploy using Bazel, or a discussion starter, send it to bazel-discuss@googlegroups.com
  • if this is a bug or feature request, fill the form below as best as you can.

Description of the problem / feature request:

One of our customers running Bazel from remote caching of dependencies and testing is experiencing the following error when running test chunks via Bazel:

'failed: I/O exception during sandboxed execution: null' error

Feature requests: what underlying problem are you trying to solve with this feature?

Ensuring that these tests run in a stable fashion and not fail intermittently

Bugs: what's the simplest, easiest way to reproduce this bug? Please provide a minimal example if possible.

Run bazel test //com/dummy:target using GCP remote caching.

What operating system are you running Bazel on?

Ubuntu 16.04

What's the output of bazel info release?

release 1.2.0

If bazel info release returns "development version" or "(@non-git)", tell us how you built Bazel.

N/A

What's the output of git remote get-url origin ; git rev-parse master ; git rev-parse HEAD ?

N/A

Have you found anything relevant by searching the web?

I looked in GitHub issues: 'https://github.com/bazelbuild/bazel/issues' but this did not give an exact match for the issue.

Any other information, logs, or outputs that you want to share?

/home/brianoh/Documents/Document with log entries for Bazel Github case.odt

15:12:33 I: Started docker-compose run -e BASIC_OUTPUT=1 -e GOOGLE_STORAGE_CREDENTIALS -v /tmp/test-chunk-artifacts/<customer repo name>tests_bazel_test_chunks___services_ads_src_test_scala_com_<customer name><customer repo name>adServer_ad_server_test_suite_AdServerUniversalTagTest_scala/try1/container-artifacts:/tmp/artifacts test-task test run --timeout=1800 <customer repo name>tests/bazel-test.chunks///services/ads/src/test/scala/com/<customer name>/<customer repo name>/adServer:ad_server_test_suite_AdServerUniversalTagTest.scala 15:12:34 E: Starting project<customer repo name>-image_1_a1cfce47b369 ... 15:12:34 E: Starting project<customer repo name>-dev-storage-image_1_eee29f413682 ... 15:12:35 E:  Starting project_<customer repo name>-dev-storage-image_1_eee29f413682 ... done Starting project_temp-dev-storage_1_64035859b922 ... 15:12:38 O: ------> Running chunk '<customer repo name>tests/bazel-test.chunks///services/ads/src/test/scala/com/<customer name>/<customer repo name>/adServer:ad_server_test_suite_AdServerUniversalTagTest.scala' with timeout 1800 seconds 15:12:38 O: Extracting Bazel installation... 15:12:41 O: Starting local Bazel server and connecting to it... 15:12:41 O: WARNING: ignoring LD_PRELOAD in environment. 15:12:44 O: INFO: Invocation ID: e7453d10-6a12-4ed7-88bb-d06e5ff8189b 15:12:44 O: Loading: 15:12:44 O: Loading: 0 packages loaded 15:12:45 O: Loading: 0 packages loaded 15:12:46 O: Loading: 0 packages loaded 15:12:47 O: Loading: 0 packages loaded 15:12:53 O: Loading: 0 packages loaded 15:12:54 O: Loading: 0 packages loaded 15:12:55 O: Analyzing: target //services/ads/src/test/scala/com/<customer name>/<customer repo name>/adServer:ad_server_test_suite_AdServerUniversalTagTest.scala (1 packages loaded, 0 targets configured) 15:12:57 O: Analyzing: target //services/ads/src/test/scala/com/<customer name>/<customer repo name>/adServer:ad_server_test_suite_AdServerUniversalTagTest.scala (11 packages loaded, 6 targets configured) 15:12:59 O: Analyzing: target //services/ads/src/test/scala/com/<customer name>/<customer repo name>/adServer:ad_server_test_suite_AdServerUniversalTagTest.scala (88 packages loaded, 217 targets configured) 15:13:02 O: Analyzing: target //services/ads/src/test/scala/com/<customer name>/<customer repo name>/adServer:ad_server_test_suite_AdServerUniversalTagTest.scala (94 packages loaded, 240 targets configured) 15:13:04 O: Analyzing: target //services/ads/src/test/scala/com/<customer name>/<customer repo name>/adServer:ad_server_test_suite_AdServerUniversalTagTest.scala (114 packages loaded, 926 targets configured) 15:13:07 O: Analyzing: target //services/ads/src/test/scala/com/<customer name>/<customer repo name>/adServer:ad_server_test_suite_AdServerUniversalTagTest.scala (148 packages loaded, 1236 targets configured) 15:13:11 O: Analyzing: target //services/ads/src/test/scala/com/<customer name>/<customer repo name>/adServer:ad_server_test_suite_AdServerUniversalTagTest.scala (186 packages loaded, 1390 targets configured) 15:13:15 O: Analyzing: target //services/ads/src/test/scala/com/<customer name>/<customer repo name>/adServer:ad_server_test_suite_AdServerUniversalTagTest.scala (231 packages loaded, 1600 targets configured) 15:13:20 O: Analyzing: target //services/ads/src/test/scala/com/<customer name>/<customer repo name>/adServer:ad_server_test_suite_AdServerUniversalTagTest.scala (287 packages loaded, 2318 targets configured) 15:13:23 O: INFO: Analyzed target //services/ads/src/test/scala/com/<customer name>/<customer repo name>/adServer:ad_server_test_suite_AdServerUniversalTagTest.scala (311 packages loaded, 2559 targets configured). 15:13:23 O: INFO: Found 1 test target... 15:13:23 O: [0 / 121] [Prepa] BazelWorkspaceStatusAction stable-status.txt 15:13:29 O: [96 / 627] Compiling external/com_google_protobuf/src/google/protobuf/text_format.cc [for host]; 0s remote-cache ... (2 actions, 1 running) 15:13:36 O: [214 / 627] Compiling external/com_google_protobuf/src/google/protobuf/extension_set_heavy.cc [for host]; 0s remote-cache ... (2 actions, 1 running) 15:13:44 O: [709 / 1,002] Stamping manifest of @maven//:com_fasterxml_jackson_module_jackson_module_paranamer; 0s remote-cache ... (2 actions running) 15:13:53 O: [1,074 / 1,165] scala //shared/scala/service-common/src/main/scala/com/<customer name>/<customer repo name>/finagle/distributedTrace:distributedTrace; 0s remote-cache ... (2 actions, 1 running) 15:14:02 O: ERROR: /app/services/ads/src/test/scala/com/<customer name>/<customer repo name>/adServer/BUILD:142:1: failed: I/O exception during sandboxed execution: null 15:14:02 O: Target //services/ads/src/test/scala/com/<customer name>/<customer repo name>/adServer:ad_server_test_suite_AdServerUniversalTagTest.scala up-to-date: 15:14:02 O: /cache/bazel/output_base/execroot/<customer repo name>/bazel-out/k8-fastbuild/bin/services/ads/src/test/scala/com/<customer name>/<customer repo name>/adServer/ad_server_test_suite_AdServerUniversalTagTest.scala 15:14:02 O: /cache/bazel/output_base/execroot/<customer repo name>/bazel-out/k8-fastbuild/bin/services/ads/src/test/scala/com/<customer name>/<customer repo name>/adServer/ad_server_test_suite_AdServerUniversalTagTest.scala.jar 15:14:02 O: INFO: Elapsed time: 83.076s, Critical Path: 8.47s 15:14:02 O: INFO: 753 processes: 753 remote cache hit. 15:14:02 O: FAILED: Build did NOT complete successfully 15:14:02 O: //services/ads/src/test/scala/com/<customer name>/<customer repo name>/adServer:ad_server_test_suite_AdServerUniversalTagTest.scala NO STATUS 15:14:02 O: 15:14:02 O: Executed 0 out of 1 test: 1 was skipped. 15:14:02 O: FAILED: Build did NOT complete successfully 15:14:03 E:  Starting project_temp-dev-storage_1_64035859b922 ... done  Starting project_<customer repo name>-image_1_a1cfce47b369 ... done 

closed time in a month

brianoh1979

push eventjmmv/endbasic

Deployment Bot (from Travis CI)

commit sha e21371809eb83527b7b2f74043af7691f21a7faa

Deploy jmmv/endbasic to github.com/jmmv/endbasic.git:gh-pages

view details

push time in a month

pull request commentbazelbuild/bazel

Add transition support for Apple bitcode

This feels like a clutch but I know very little about this to tell.

Is watchOS the only place where we need this? Does the flag have to hardcode the name or it could be made more general? Wondering if we'll end up having to add extra flags for iOS, tvOS, etc. in the future.

Also, why are we making this conditional on -c opt? In other words, couldn't this be --watchos_force_bitcode and just do the thing no matter what the compilation mode is, relying on the user to only pass this for release builds? Or would that not work?

thii

comment created time in a month

push eventjmmv/endbasic

Deployment Bot (from Travis CI)

commit sha 59335bfd43c8e0a5d320002a0fbc970ffccddd3e

Deploy jmmv/endbasic to github.com/jmmv/endbasic.git:gh-pages

view details

push time in a month

delete branch jmmv/endbasic

delete branch : clippy

delete time in a month

push eventjmmv/endbasic

Deployment Bot (from Travis CI)

commit sha 01fe92b4fc8ff128c21e3fa4a4aea49e78b613be

Deploy jmmv/endbasic to github.com/jmmv/endbasic.git:gh-pages

view details

push time in a month

push eventjmmv/endbasic

Julio Merino

commit sha 68efdbc12fcd22b04419c6b11d809e75fca5529f

Address some extra redundant_field_names warnings

view details

push time in a month

PR merged jmmv/endbasic

Address some extra redundant_field_names warnings
+3 -3

0 comment

2 changed files

jmmv

pr closed time in a month

delete branch jmmv/sandboxfs

delete branch : clippy

delete time in a month

create barnchjmmv/sandboxfs

branch : clippy

created branch time in a month

PR opened jmmv/endbasic

Address some extra redundant_field_names warnings
+3 -3

0 comment

2 changed files

pr created time in a month

push eventjmmv/endbasic

Julio Merino

commit sha 1f3e3852075ad73ac1bd12d8d5530878b71aaefc

Hide the scrollbar presented by xterm.js I'm not sure this is the best way to go but it seems to do the trick. I'm also not disabling scrollback, which is still accessible via the mouse, because it seems useful in some cases and it doesn't hurt if it's present but out of the way.

view details

Julio Merino

commit sha 36eddf443326d24f9779eaa030373d5f9c872b61

Reduce the font on the web a little bit This setting should be customizable in some way, but for now, the previous value seemed too big in general.

view details

Julio Merino

commit sha 735e5a4d91a2b2dde714517d2094486e58ef5aab

Point to my blog posts from the front page

view details

Julio Merino

commit sha e18a8738464355c402e49484e210214a1554bf2e

Address some extra redundant_field_names warnings

view details

push time in a month

create barnchjmmv/endbasic

branch : clippy

created branch time in a month

push eventjmmv/endbasic

Deployment Bot (from Travis CI)

commit sha 6bc432f4ca2fb22595a3495d82065a1bade783e6

Deploy jmmv/endbasic to github.com/jmmv/endbasic.git:gh-pages

view details

push time in a month

push eventjmmv/endbasic

Deployment Bot (from Travis CI)

commit sha ea6c6fb9728c1a842f24d977271cbb3c24ad9159

Deploy jmmv/endbasic to github.com/jmmv/endbasic.git:gh-pages

view details

push time in a month

push eventjmmv/endbasic

Deployment Bot (from Travis CI)

commit sha 04ff92a3c5d511cb7343375024b0fc22d973d6aa

Deploy jmmv/endbasic to github.com/jmmv/endbasic.git:gh-pages

view details

push time in a month

push eventjmmv/endbasic

Deployment Bot (from Travis CI)

commit sha d238e6d39b9e7cd459c18e40203a1768ceca0d7d

Deploy jmmv/endbasic to github.com/jmmv/endbasic.git:gh-pages

view details

push time in a month

push eventjmmv/endbasic

Deployment Bot (from Travis CI)

commit sha 76dceb095340f531a670e9bb0ec63ca54e569fbe

Deploy jmmv/endbasic to github.com/jmmv/endbasic.git:gh-pages

view details

push time in a month

push eventjmmv/endbasic

Deployment Bot (from Travis CI)

commit sha 9afb95d167a997f90330d374a83fbfd8c9e21bfa

Deploy jmmv/endbasic to github.com/jmmv/endbasic.git:gh-pages

view details

push time in a month

push eventjmmv/endbasic

Deployment Bot (from Travis CI)

commit sha d624b5bcef2a30df00edf066ce5d6ffd3a5c1730

Deploy jmmv/endbasic to github.com/jmmv/endbasic.git:gh-pages

view details

push time in 2 months

push eventjmmv/endbasic

Deployment Bot (from Travis CI)

commit sha 3015afb080bb849e8547a09b15332f236afcd693

Deploy jmmv/endbasic to github.com/jmmv/endbasic.git:gh-pages

view details

push time in 2 months

push eventjmmv/endbasic

Deployment Bot (from Travis CI)

commit sha b71db271b9f1cd9d543ca5366c6a78417ca733ee

Deploy jmmv/endbasic to github.com/jmmv/endbasic.git:gh-pages

view details

push time in 2 months

push eventjmmv/endbasic

Deployment Bot (from Travis CI)

commit sha b3ab8d5e4617bb1c1d9da7daee477b104f009376

Deploy jmmv/endbasic to github.com/jmmv/endbasic.git:gh-pages

view details

push time in 2 months

push eventjmmv/endbasic

Deployment Bot (from Travis CI)

commit sha e2b6040b1d021a3d1892fa92313b12bda63fd0e0

Deploy jmmv/endbasic to github.com/jmmv/endbasic.git:gh-pages

view details

push time in 2 months

issue commentbazelbuild/bazel

process-wrapper-legacy.cc: error: 'PR_SET_CHILD_SUBREAPER' was not declared in this scope

I mean, just defining PR_SET_CHILD_SUBREAPER to a random value and making prctl fail, without having a fallback, wouldn't work because we now take advantage of the subreaper feature to wait for processes.

And if we have a fallback (which we still do), then it seems better to #ifdef that code out behind the presence of PR_SET_CHILD_SUBREAPER instead of redefining the constant.

beasleyr-vmw

comment created time in 2 months

issue commentbazelbuild/bazel

process-wrapper-legacy.cc: error: 'PR_SET_CHILD_SUBREAPER' was not declared in this scope

Defining PR_SET_CHILD_SUBREAPER ourselves would likely break the new process waiting logic. But we still have the old logic around behind the wait_fix checks... so we could keep it indefinitely and enable it once this feature isn't present. Not my preference as I was hoping to delete the old slightly broken logic... but should be an easy change at this point given that it's still present.

beasleyr-vmw

comment created time in 2 months

issue commentbazelbuild/bazel

process-wrapper-legacy.cc: error: 'PR_SET_CHILD_SUBREAPER' was not declared in this scope

I think this is a question for @philwo. Do we have to support CentOS 6 out of the box?

beasleyr-vmw

comment created time in 2 months

push eventjmmv/endbasic

Deployment Bot (from Travis CI)

commit sha 55c238befd05732ab0da8e52a21bcbbd69c795ac

Deploy jmmv/endbasic to github.com/jmmv/endbasic.git:gh-pages

view details

push time in 2 months

issue closedbazelbuild/bazel

bazel fetches fail randomly on Kokoro + macOS

I got a bazel error that told me to file an issue here.

Description of the problem / feature request:

Our CI build, running on macOS in Kokoro. The build log is here.

Here's a snippet of the relevant bazel error:

ERROR: An error occurred during the fetch of repository 'local_config_cc':
   Traceback (most recent call last):
	File "/Volumes/BuildData/tmpfs/tmp/bazel/external/bazel_tools/tools/cpp/cc_configure.bzl", line 123
		configure_osx_toolchain(<2 more arguments>)
	File "/Volumes/BuildData/tmpfs/tmp/bazel/external/bazel_tools/tools/cpp/osx_cc_configure.bzl", line 138, in configure_osx_toolchain
		fail(<1 more arguments>)
wrapped_clang failed to generate. Please file an issue at https://github.com/bazelbuild/bazel/issues with the following:
return code 256, stderr: Timed out, stdout:
INFO: Call stack for the definition of repository 'remote_java_tools_darwin' which is a http_archive (rule definition at /Volumes/BuildData/tmpfs/tmp/bazel/external/bazel_tools/tools/build_defs/repo/http.bzl:292:16):
 - /Volumes/BuildData/tmpfs/tmp/bazel/external/bazel_tools/tools/build_defs/repo/utils.bzl:205:9
 - /DEFAULT.WORKSPACE.SUFFIX:280:1

Bugs: what's the simplest, easiest way to reproduce this bug? Please provide a minimal example if possible.

No easy way to reproduce. We occasionally see errors like this on our Kokoro CI builds (usually? always? on macOS).

What operating system are you running Bazel on?

macOS. not sure what version.

What's the output of bazel info release?

Maybe this:

Build label: 2.0.0
Build target: bazel-out/darwin-opt/bin/src/main/java/com/google/devtools/build/lib/bazel/BazelServer_deploy.jar
Build time: Thu Dec 19 12:33:30 2019 (1576758810)
Build timestamp: 1576758810
Build timestamp as int: 1576758810

What's the output of git remote get-url origin ; git rev-parse master ; git rev-parse HEAD ?

Our repo is here: https://github.com/googleapis/google-cloud-cpp

The issue happens on any number of commits, but not consistently. It happens on our Kokoro CI runs.

Have you found anything relevant by searching the web?

No.

Any other information, logs, or outputs that you want to share?

The full error log: https://source.cloud.google.com/results/invocations/d77fd19f-24ea-46c0-a3c1-da34656f63be/targets/cloud-cpp%2Fgithub%2Fgoogle-cloud-cpp%2Fmaster%2Fmacos%2Fbazel/log

closed time in 2 months

devjgm

issue commentbazelbuild/bazel

bazel fetches fail randomly on Kokoro + macOS

Closing due to lack of information. Please reply if there is any chance you'd try what was asked in the last reply, because if that works, we'd fix this rather easily.

devjgm

comment created time in 2 months

issue commentbazelbuild/bazel

Pulling images from GCR fails if gcloud auth helper is in use

Given that --spawn_strategy=standalone fixes the problem, this sounds like the sandbox at play. And it makes sense: actions shouldn't be allowed to poke at your home directory freely, and the sandbox denies that.

What exact command are you running? I don't have a lot of context on how we interact with Docker, so I'm wondering where that gcloud call comes from.

teran

comment created time in 2 months

more