Skip to content
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.

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

nerves.system.shell information out of order #921

Open
LostKobrakai opened this issue Oct 26, 2023 · 1 comment
Open

nerves.system.shell information out of order #921

LostKobrakai opened this issue Oct 26, 2023 · 1 comment

Comments

@LostKobrakai
Copy link

LostKobrakai commented Oct 26, 2023

Environment

  • Elixir version (elixir -v): 1.15.6
  • Nerves environment: (mix nerves.env --info)
|nerves_bootstrap| Environment Package List

  Pkg:         nerves_system_da_1000
  Vsn:         0.0.1
  Type:        system
  BuildRunner: {Nerves.Artifact.BuildRunners.Local, [make_args: ["source", "all", "legal-info"]]}

  Pkg:         nerves_system_br
  Vsn:         1.24.1
  Type:        system_platform
  BuildRunner: {nil, []}

  Pkg:         nerves_toolchain_x86_64_nerves_linux_musl
  Vsn:         1.8.0
  Type:        toolchain
  BuildRunner: {Nerves.Artifact.BuildRunners.Local, []}

  Pkg:         nerves_toolchain_ctng
  Vsn:         1.9.3
  Type:        toolchain_platform
  BuildRunner: {nil, []}

|nerves_bootstrap| Loadpaths Start

|nerves_bootstrap| Precompile Start

==> nerves_toolchain_x86_64_nerves_linux_musl
|nerves_bootstrap| Compile.NervesPackage start

|nerves_bootstrap| Compile.NervesPackage end

==> nerves_system_da_1000
Generated nerves_system_da_1000 app
|nerves_bootstrap| Compile.NervesPackage start

|nerves_bootstrap| Compile.NervesPackage end

|nerves_bootstrap| Precompile End


Nerves environment
  MIX_TARGET:   target
  MIX_ENV:      dev

|nerves_bootstrap| Environment Variable List
  target:     target
  toolchain:  /home/benni/.nerves/artifacts/nerves_toolchain_x86_64_nerves_linux_musl-linux_x86_64-1.8.0
  system:     /home/benni/.nerves/artifacts/nerves_system_da_1000-portable-0.0.1
  app:        .

|nerves_bootstrap| Loadpaths End
  • Additional information about your host, target hardware or environment that
    may help

Current behavior

Running the command I occationally get the result in incorrect order:

benni@jester:~/nerves-system-da-1000$ mix nerves.system.shell
==> nerves
==> nerves_system_da_1000
|nerves_bootstrap| shell start deprecated
  /home/benni/nerves-system-da-1000/deps/nerves_system_br/create-build.sh /home/benni/nerves-system-da-1000/nerves_defconfig /home/benni/nerves-system-da-1000/.nerves/artifacts/nerves_system_da_1000-portable-0.0.1 >/dev/null && cd /home/benni/nerves-system-da-1000/.nerves/artifacts/nerves_system_da_1000-portable-0.0.1
OTP 26 made several changes to the serial interface handling. Unfortunately, this
is a regression in preventing the Nerves tooling from starting a system sub-shell.

However, compilation is supported on this host and this native shell can be used.
Run the commands below to create the build directory and perform all the same
interactions as before within it:

Expected behavior

Proper order of information.

@jjcarstens
Copy link
Member

I've investigated this a little bit, but don't have any more leads. The final command is in its own log statement so that it could be shoved to stdin for the hacky workarounds. I thought it my be due to logger being async and out of order, but @LostKobrakai has also discovered some issues with the shell duplicating messages when using SSH and NervesHub

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

No branches or pull requests

2 participants