Skip to content

tests: Add shbang to python scripts with a main#8430

Merged
randall merged 1 commit intoapache:masterfrom
randall:add_shbang
Oct 18, 2021
Merged

tests: Add shbang to python scripts with a main#8430
randall merged 1 commit intoapache:masterfrom
randall:add_shbang

Conversation

@randall
Copy link
Copy Markdown
Contributor

@randall randall commented Oct 18, 2021

No description provided.

@randall randall added the Tests label Oct 18, 2021
@randall randall added this to the 10.0.0 milestone Oct 18, 2021
@randall randall requested a review from bneradt October 18, 2021 20:46
@randall randall self-assigned this Oct 18, 2021
@randall randall requested a review from bryancall as a code owner October 18, 2021 20:46
@bneradt
Copy link
Copy Markdown
Contributor

bneradt commented Oct 18, 2021

This is a good change and shouldn't hurt anything. It could help with one-off executions of these scripts, for instance.

Please be aware, however, that all these scrips should be executed using the env.executable Python path so that they run with the same Python AuTest runs with:
#8412

That makes it easy to configure the environment needed for these scripts: simply update tests/Pipfile.

Thanks!

@randall
Copy link
Copy Markdown
Contributor Author

randall commented Oct 18, 2021

It was in the one off case I hit this and figured someone else would try to do the same in the future.

@randall randall merged commit 727106e into apache:master Oct 18, 2021
@randall randall deleted the add_shbang branch October 18, 2021 22:59
zwoop pushed a commit that referenced this pull request Nov 8, 2021
@zwoop
Copy link
Copy Markdown
Contributor

zwoop commented Nov 8, 2021

Cherry-picked to v9.2.x

@zwoop zwoop modified the milestones: 10.0.0, 9.2.0 Nov 8, 2021
moonchen pushed a commit to moonchen/trafficserver that referenced this pull request Mar 17, 2022
* asf/9.2.x: (50 commits)
  Updated ChangeLog
  Reject Transfer-Encoding in pre-HTTP/1.1 requests (apache#8451)
  Better TLS Secrets Truncation. (apache#8489)
  ssl_secret debug printing: print only the first 50 bytes (apache#8483)
  Define TS_HTTP_VALUE_BROTLI and TS_HTTP_LEN_BROTLI (apache#8477)
  Fix case of brotli (apache#8476)
  TSSslSecretSet: Update SSL_CTX TLS Secrets (apache#8368)
  Adding doc/README.md (apache#8420)
  Doc: fix typos in Strategy documentation (apache#8408)
  Refactors and promotes the Txn Control mechanism with Get() and Set() (apache#8428)
  tests: Add shbang to python scripts with a main (apache#8430)
  Remove empty tests/unit_tests directoy+makefile (apache#8429)
  Adds new API: TSVConnSslSniGet (apache#8313)
  rate_limit: convert to using TSVConnSslSniGet (apache#8414)
  Update the Multiplexer Docs for Multplexed HTTPS Connections (apache#8440)
  bigobj: use automake to build test utilities (apache#8441)
  Make sni.yaml errors cause an unrecoverable TS crash at startup. (apache#8208)
  Fix timeout checks of NetHandler::manage_active_queue() (apache#8287)
  Fix Multiplexer POST/PUT Body Handling (apache#8439)
  Document proxy.config.memory.max_usage (apache#8450)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants