Age | Commit message (Collapse) | Author |
|
- Bad dynamic cast target ...classic
- std::shared_ptr need explicit deref
|
|
add-body-to-network-errors
|
|
We have a larger problem that passsing computed strings to the first
variable argument of many exception constructors is unsafe because that
first variable argument is interpreted not as a plain string, but format
string, and if it contains '%' boost::format will abort, since there are
no arguments to the format string.
In this particular instance '%' was used as part of an escape code in a
URL, which, when the download failed, caused Nix to abort displaying the
`FileTransferError`.
|
|
Mention number of derivations to be build/fetched in output
|
|
|
|
(cherry picked from commit 0a1d3c1dd311f94e9d1f56e1aa7fe1ab34314ec1)
|
|
|
|
(cherry picked from commit aa0e2a2e70a3519a9dcb9b1da000a13c01aa6cc1)
|
|
See <https://github.com/NixOS/nix/pull/3710#issuecomment-645480333>
|
|
|
|
Everywhere seems to use `getHTTPStatus` now.
|
|
This is a bit complex because we want to expose extra functionality the
wrapped class has. Perhaps there is some inheritancy trickery to do this
nicer, but I don't know it, and this is the first thing we tried after a
series of attempts that did build.
This design is kind of like that of Rust's Writer, Reader, or Iter
adapters, which impliment more traits based on what the inner type
implements.
|
|
This was introduced in fa125b9b28bea25a4eeb4d39a71a481563127cb9, and
then "reverted" in 1cf480110879ffc8aee94b4b75999da405b71d7c, except that
revert left the struct around doing nothing useful.
We're removing it all the way now because we want to make a new
`TeeSink` complementing the already-exiting `TeeSource`, that is
actually a completely different concept as far as the class hierarchy is
concerned.
|
|
Cleanup class StorePath warning
|
|
obsidiansystems/make-http-successful-states-coherent
Make successful states coherent
|
|
|
|
also a similar case with struct Goal
|
|
|
|
Reserve the `__contentAddressed` derivation parameter
|
|
replaced uncaught_exception with uncaught_exceptions
|
|
|
|
The successful states used in these two places in the code were slightly
different. Should they be the same list?
|
|
|
|
Not implementing anything here, just throwing an error if a derivation
sets `__contentAddressed = true` without
`--experimental-features content-addressed-paths`
(and also with it as there's nothing implemented yet)
|
|
obsidiansystems/sligthly-improve-store-path-documentation
Clarify the description of StorePath inputs
|
|
Also correct grammar for the case of a single derivation.
|
|
|
|
|
|
|
|
Rewrite StorePath class in C++
|
|
|
|
On nix-env -qa -f '<nixpkgs>', this reduces maximum RSS by 20970 KiB
and runtime by 0.8%. This is mostly because we're not parsing the hash
part as a hash anymore (just validating that it consists of base-32
characters).
Also, replace storePathToHash() by StorePath::hashPart().
|
|
Fixes #3618.
|
|
|
|
|
|
E.g. instead of
error: --- BuildError ----------------------------------------------- nix
builder for '/nix/store/03nk0a3n8h2948k4lqfgnnmym7knkcma-foo.drv' failed with exit code 1
error: --- Error ---------------------------------------------------- nix
build of '/nix/store/03nk0a3n8h2948k4lqfgnnmym7knkcma-foo.drv' failed
we now get
error: --- Error ---------------------------------------------------- nix
builder for '/nix/store/03nk0a3n8h2948k4lqfgnnmym7knkcma-foo.drv' failed with exit code 1
|
|
|
|
|
|
|
|
|
|
|
|
|
|
These are now shown in the progress bar.
Closes #3577.
|
|
|
|
|
|
This means that 'throw Error({ ... ErrorInfo ... })' now works.
|
|
|
|
|
|
|
|
Allow empty hash in derivations
|