Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Source sys and data configs #2496

Closed
wants to merge 1 commit into from
Closed

Conversation

davispuh
Copy link

Source config files from paths like /etc/fish/*.fish for sysadmins and /usr/share/fish/*.fish for distributions/vendors.
This basically implements #1956

@faho
Copy link
Member

faho commented Oct 20, 2015

Please see my comment in #1956.

One thing you don't do here is the precedence system - files in $__fish_sysconfdir should override files in $__fish_datadir (I'd say they should even if they are empty or unreadable) and those in $XDG_CONFIG_HOME/fish should take precedence over both.

Another is that this doesn't implement "vendor" directories for eagerly loaded files (to put their event handlers in).

The third point is it doesn't handle the lazy-loaded ("function") part at all.

Now, I'm not saying that my proposal is the end-all-be-all, but this is certainly a lot simpler. In my opinion, it's too simple.

@davispuh
Copy link
Author

I agree with all what you said, but currently we don't have anything working so I thought I'll make this quickly while someone implements it properly :P That issue have been open since Feb and no one have come up with code.

It's so simple, because it was really quick to implement this and get something working atleast. Of course I agree that proper implementation would be better but I don't know when I'll have time for that.

@faho
Copy link
Member

faho commented Oct 20, 2015

I see where you're coming from, I just don't think a simple stop-gap solution helps in this case. For one, this is an interface for other projects to rely on - so if they start using the stop-gap, we must either keep things that way or ask others to switch again.

For a competing mock-proposal, see #2498.

@krader1961
Copy link
Contributor

I'm happy to see a concrete implementation to further the discussion but don't think this should be merged in its present state. One of the reasons I decided to abandon zsh was that too many changes like this one were adopted without a discussion of the pros and cons of the proposed solution and the alternatives. The seductive pull of "this solution isn't optimal but it sorta kinda works and no better solution has been presented" needs to be resisted other than as a way to drive the discussion forward.

@zanchey
Copy link
Member

zanchey commented Dec 11, 2015

I definitely prefer the approach in #2498, so I'm going to close this for now - but thanks for getting the conversation moving!

@zanchey zanchey closed this Dec 11, 2015
@zanchey zanchey self-assigned this Dec 11, 2015
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Apr 17, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants