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

update pill 7.2 with nix log, add clarification #146

Closed
wants to merge 1 commit into from

Conversation

ammkrn
Copy link

@ammkrn ammkrn commented May 30, 2020

In pill 7.2, the terminal output of nix-repl> :b d no longer includes the build logs, which makes the exercise in 7.2 very difficult to understand. As tomberek on the discourse pointed out, the logs showing the output of declare -xp should be viewed with nix log <storepath>. I also added a little bit of explanation to hopefully make it more clear what's happening, and why we're using declare -xp to print variables.

Copy link

@jfly jfly left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not a maintainer, but just ran into similar confusion myself. Would love to see this get merged up!


this derivation produced the following outputs:
out -> /nix/store/gczb4qrag22harvv693wwnflqy7lx5pb-<emphasis>foo</emphasis></computeroutput></screen>
declare -x system="x86_64-linux"</computeroutput></screen>
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

uber nit: This file used to end in a newline, it looks like it doesn't anymore. No idea if that actually matters =)

jfly added a commit to jfly/nix that referenced this pull request Jun 2, 2021
While working my way through the nix-pills, I was surprised to not see
the stdout of a build when realising a derivation. I eventually
discovered `nix log` and NixOS/nix-pills#146,
but the *first* place I looked was this man page expecting to see a
note about stdout/stderr getting suppressed during builds.
Unfortunately, I didn't find anything. I think this addition to the man
page would help future learners like myself.
jfly added a commit to jfly/nix that referenced this pull request Jun 2, 2021
While working my way through the nix-pills, I was surprised to not see
the stdout of a build when realising a derivation. I eventually
discovered `nix log` and NixOS/nix-pills#146,
but the *first* place I looked was this man page expecting to see a
note about stdout/stderr getting suppressed during builds.
Unfortunately, I didn't find anything. I think this addition to the man
page would help future learners like myself.
@drupol
Copy link
Contributor

drupol commented Oct 18, 2023

Is this issue still relevant?

@jfly
Copy link

jfly commented Oct 19, 2023

@drupol, no, it looks like this was resolved in #174

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.

None yet

4 participants