Skip to content

Conversation

silverjam
Copy link
Contributor

@silverjam silverjam commented Aug 1, 2022

Cherry pick of #654

* Disable attach_console unless indicated

* disable attach_console in server code too...
@silverjam silverjam changed the title Disable attach_console unless indicated [CPP-792] (#654) Disable attach_console unless indicated [CPP-792] [v4.0] Aug 1, 2022
@silverjam
Copy link
Contributor Author

@john-michaelburke this PR shows a SWIFTNAV_CONSOLE_DEBUG that we can re-use for the debug env dump too

@john-michaelburke
Copy link
Collaborator

Should we merge this to main first?

@silverjam silverjam enabled auto-merge (squash) August 1, 2022 19:55
@silverjam
Copy link
Contributor Author

Should we merge this to main first?

It's already on main, I forgot to backport it

@john-michaelburke
Copy link
Collaborator

Might need to backport this commit as well:
52f213d

@silverjam silverjam merged commit cf936bc into swift-toolbox-v4.0.0-branch Aug 2, 2022
@silverjam silverjam deleted the silverjam/cherry-pick/47621130637ffd972c334e0c296034b16d0d8ff0 branch August 2, 2022 02:01
@silverjam
Copy link
Contributor Author

Might need to backport this commit as well: 52f213d

I eventually got to pass by retrying enough time... for future backports I cherry-picked this commit directly and pushed it (bypassed the PR process).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants