starseeker (May 14 2022 at 13:23): starseeker (May 14 2022 at 13:23):@Sean seeing a build failure in a clean tree:
Generating pkgIndex.tcl in /home/user/brlcad/build/./share/tclscripts/archer
[snip]
can't read "::ArcherCore::inheritFromToplevel": no such variable
Unexpected error encountered while running Archer.
Aborting.
Okay @starseeker sorry about that! I’ll be back at a computer in a couple hours and will look at it. That’s strange.. doesn’t parse into the subdir here.
Erik (May 14 2022 at 16:50):I saw that yesterday, blew away my build dir and started over and it was fine. Just did a ninja clean, rebuilt, not seeing it
starseeker (May 14 2022 at 19:57):OK, I'll try again
starseeker (May 14 2022 at 20:34):Yep, worked this time. I must have cleaned another directory or something. Sorry for the noise @Sean
Sean (May 14 2022 at 21:05):ahh, okay -- I was going to say I was having trouble reproducing it, but was working on setting up a linux host to test on
Sean (May 14 2022 at 21:06):its undoubtedly because the build rule was simply copying archer_laucher.tcl into the dir, so when I moved it to a subdir .. it's still sitting there copied into the share/tclscripts/archer folder, so when mk_pkgIndex runs, it still picks it up.
starseeker (May 15 2022 at 00:22):Ah, so the stale symlink cleanup we use for most of that sort of thing wouldn't have caught the copy. Makes sense.
starseeker (May 15 2022 at 00:24):Acting more like a Windows build for that specific case. I must have done the copy in the early stages of the CMake conversion because I figured that file was unlikely to change enough to warrant the (at the time) added complexity. Whoops.
Sean (May 15 2022 at 02:15):that's also why I moved it into a subdir. it's always been throwing an error into the output. I addressed the original issue, and a half dozen others, but the the protections for mk_pkgIndex started to get too deep as it's parsing a run script like a package script and that's the underlying issue. now should be fixed proper though.
Last updated: Oct 09 2024 at 00:44 UTC