Skip to content

Conversation

ferdymercury
Copy link
Collaborator

@ferdymercury ferdymercury commented Sep 25, 2025

@hageboeck hageboeck self-assigned this Sep 25, 2025
@hageboeck hageboeck added the skip ci Skip the full builds on the actions runners label Sep 25, 2025
Copy link
Member

@hageboeck hageboeck left a comment

Choose a reason for hiding this comment

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

Hello, great initiative! I believe I found a hack to do this in a two-liner. I tested locally with the original thisroot.sh and with a link, and both resulted in what I expected.

Copy link

github-actions bot commented Sep 25, 2025

Test Results

0 tests   0 ✅  0s ⏱️
0 suites  0 💤
0 files    0 ❌

Results for commit 7d3329b.

♻️ This comment has been updated with latest results.

@ferdymercury
Copy link
Collaborator Author

ferdymercury commented Sep 25, 2025

In fish, it's:

set thisroot (path dirname $SOURCE)
set -xg ROOTSYS (set oldpwd $PWD; cd $thisroot/.. > /dev/null;pwd;cd $oldpwd; set -e oldpwd)

In csh, it's:

set thisroot="`dirname ${thisfile}`"
setenv ROOTSYS "`(cd ${thisroot}/..;pwd)`"

But I need help here on how to change it

Copy link
Member

@hageboeck hageboeck left a comment

Choose a reason for hiding this comment

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

LGTM
It's a very good idea to check for root-config, because that will detect most errors!

@ferdymercury ferdymercury marked this pull request as draft September 25, 2025 12:55
@ferdymercury ferdymercury marked this pull request as ready for review September 29, 2025 11:57
@ferdymercury ferdymercury marked this pull request as draft September 29, 2025 11:58
@hageboeck
Copy link
Member

@ferdymercury we must have made a mistake during merging. I believe that this PR ferdymercury#11 fixes it.

@ferdymercury
Copy link
Collaborator Author

@ferdymercury we must have made a mistake during merging. I believe that this PR ferdymercury#11 fixes it.

Thanks a lot! Seems fixed now.

I'll leave it as "Draft" while we think on how to deal with #19974 (comment)

@ferdymercury ferdymercury marked this pull request as ready for review September 30, 2025 09:57
@hageboeck hageboeck merged commit ef1b534 into root-project:master Oct 3, 2025
8 checks passed
@ferdymercury ferdymercury deleted the patch-12 branch October 3, 2025 09:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
skip ci Skip the full builds on the actions runners
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants