puck/lix
0
0
Fork 0
forked from lix-project/lix
Code Pull requests Activity

Rename yellowtxt -> magentatxt

Browse source 
`yellowtxt` wraps its value with `ANSI_WARNING`, but `ANSI_WARNING` has
been equal to `ANSI_MAGENTA` for a long time. Now the name is updated.
This commit is contained in:
Rebecca Turner 2024-02-03 19:16:30 -08:00
parent 49cf090cb2
commit a7939a6c2a
Signed by: rbt
SSH key fingerprint: SHA256:HU9KTlGeHEBIp12MT4UHG0uc//NRNoJMWfzVGX3igu4
3 changed files with 9 additions and 11 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
src/libstore/build/derivation-goal.cc
View file

@ -708,7 +708,7 @@ void DerivationGoal::tryToBuild()
if (!outputLocks.lockPaths(lockFiles, "", false)) { if (!outputLocks.lockPaths(lockFiles, "", false)) {
if (!actLock) if (!actLock)
actLock = std::make_unique<Activity>(*logger, lvlWarn, actBuildWaiting, actLock = std::make_unique<Activity>(*logger, lvlWarn, actBuildWaiting,
fmt("waiting for lock on %s", yellowtxt(showPaths(lockFiles)))); fmt("waiting for lock on %s", magentatxt(showPaths(lockFiles))));
worker.waitForAWhile(shared_from_this()); worker.waitForAWhile(shared_from_this());
return; return;
} }
@ -762,7 +762,7 @@ void DerivationGoal::tryToBuild()
the wake-up timeout expires. */ the wake-up timeout expires. */
if (!actLock) if (!actLock)
actLock = std::make_unique<Activity>(*logger, lvlWarn, actBuildWaiting, actLock = std::make_unique<Activity>(*logger, lvlWarn, actBuildWaiting,
fmt("waiting for a machine to build '%s'", yellowtxt(worker.store.printStorePath(drvPath)))); fmt("waiting for a machine to build '%s'", magentatxt(worker.store.printStorePath(drvPath))));
worker.waitForAWhile(shared_from_this()); worker.waitForAWhile(shared_from_this());
outputLocks.unlock(); outputLocks.unlock();
return; return;
@ -987,7 +987,7 @@ void DerivationGoal::buildDone()
diskFull |= cleanupDecideWhetherDiskFull(); diskFull |= cleanupDecideWhetherDiskFull();
auto msg = fmt("builder for '%s' %s", auto msg = fmt("builder for '%s' %s",
yellowtxt(worker.store.printStorePath(drvPath)), magentatxt(worker.store.printStorePath(drvPath)),
statusToString(status)); statusToString(status));
if (!logger->isVerbose() && !logTail.empty()) { if (!logger->isVerbose() && !logTail.empty()) {

2
src/libstore/build/local-derivation-goal.cc
View file

@ -232,7 +232,7 @@ void LocalDerivationGoal::tryLocalBuild()
if (!buildUser) { if (!buildUser) {
if (!actLock) if (!actLock)
actLock = std::make_unique<Activity>(*logger, lvlWarn, actBuildWaiting, actLock = std::make_unique<Activity>(*logger, lvlWarn, actBuildWaiting,
fmt("waiting for a free build user ID for '%s'", yellowtxt(worker.store.printStorePath(drvPath)))); fmt("waiting for a free build user ID for '%s'", magentatxt(worker.store.printStorePath(drvPath))));
worker.waitForAWhile(shared_from_this()); worker.waitForAWhile(shared_from_this());
return; return;
} }

12
src/libutil/fmt.hh
View file

@ -63,19 +63,17 @@ inline std::string fmt(const std::string & fs, const Args & ... args)
return f.str(); return f.str();
} }
// -----------------------------------------------------------------------------
// format function for hints in errors. same as fmt, except templated values // format function for hints in errors. same as fmt, except templated values
// are always in yellow. // are always in magenta.
template <class T> template <class T>
struct yellowtxt struct magentatxt
{ {
yellowtxt(const T &s) : value(s) {} magentatxt(const T &s) : value(s) {}
const T & value; const T & value;
}; };
template <class T> template <class T>
std::ostream & operator<<(std::ostream & out, const yellowtxt<T> & y) std::ostream & operator<<(std::ostream & out, const magentatxt<T> & y)
{ {
return out << ANSI_WARNING << y.value << ANSI_NORMAL; return out << ANSI_WARNING << y.value << ANSI_NORMAL;
} }
@ -114,7 +112,7 @@ public:
template<class T> template<class T>
hintformat & operator%(const T & value) hintformat & operator%(const T & value)
{ {
fmt % yellowtxt(value); fmt % magentatxt(value);
return *this; return *this;
} }