diff options
| -rw-r--r-- | todo.org | 9 |
1 files changed, 6 insertions, 3 deletions
@@ -280,8 +280,8 @@ Tests: =tests/safe-rm-rf/test_safe_rm_rf.py= sources the real function out of th *** TODO [#A] Standardize boolean comparison style :quick: Mixed =[ "$var" = "true" ]= vs =$var= evaluation — pick one pattern. -*** TODO [#A] Replace eval with safer alternatives :quick: -Line 434: =eval "$cmd"= — use arrays or direct execution. +*** 2026-05-26 Tue @ 15:27:09 -0500 eval task moot — the line-434 eval is gone, the survivor is deliberate +Verified: the only =eval= left in =archsetup= is line 578 in =retry_install=, and it's intentional and documented — it captures =$?= directly from =eval "$cmd"= to dodge the if-compound-swallows-exit-code trap. Replacing it with an array would reintroduce that bug. The line-434 eval this task pointed at no longer exists. Nothing to change. ** TODO [#B] Review post-archsetup laptop setup steps (velox 2026-04-10) :PROPERTIES: @@ -650,10 +650,13 @@ Establish clear process for tool evaluation decisions :END: Ensure new tools integrate with DWM environment and don't break workflow -** TODO [#B] Add Rust installation via rustup instead of pacman package :quick: +** DONE [#B] Add Rust installation via rustup instead of pacman package :quick: +CLOSED: [2026-05-26 Tue] :PROPERTIES: :LAST_REVIEWED: 2026-05-21 :END: +Already implemented — =archsetup= lines 1976-1979 (Programming Languages and Utilities) =pacman_install rustup= then =rustup default stable= as the user. Closing on verification; the task predated that work. + The =rust= package has been removed from archsetup. Need to add Rust installation using =rustup= (the official Rust toolchain manager) instead of the Arch package. Steps: |