Hi,
I think this makes more sense... I don't know why it isn't the default
https://github.com/vim/vim/pull/16309
(1 file)
—
Reply to this email directly, view it on GitHub.
You are receiving this because you are subscribed to this thread.![]()
@saccarosium pushed 1 commit.
—
View it on GitHub or unsubscribe.
You are receiving this because you are subscribed to this thread.![]()
@saccarosium pushed 1 commit.
You are receiving this because you are subscribed to this thread.![]()
thanks
—
Reply to this email directly, view it on GitHub.
You are receiving this because you are subscribed to this thread.![]()
—
Reply to this email directly, view it on GitHub.
You are receiving this because you are subscribed to this thread.![]()
Has anyone checked that this still loads appropriate syntax, ftplugins, etc.? Now I’ve got to (personally) duplicate my sh runtime files to bash, make sure ALE knows what to do when the filetype=bash, etc.
I think this is going to lead to churn for many folks :)
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.![]()
Alternatively, the default bash runtime files could runtime the shell files, which they may already?
—
Reply to this email directly, view it on GitHub.
You are receiving this because you are subscribed to this thread.![]()
Ok, should have done my homework first: the bash runtime support already redirects to the shell support. So ALE is the last place I personally need to check.
—
Reply to this email directly, view it on GitHub.
You are receiving this because you are subscribed to this thread.![]()
—
Reply to this email directly, view it on GitHub.
You are receiving this because you are subscribed to this thread.![]()
—
Reply to this email directly, view it on GitHub.
You are receiving this because you are subscribed to this thread.![]()
Sounds like both you and the submitter should do a bit more homework: this breaks in several ways, which the extant ftplugin/bash.vim is not sufficient to handle. The number of randomly broken filetypes in the last year is getting worrisome.
Perhaps you could elaborate on what else is broken? (In particular, if there's interest in building a case for reverting this, having such a list might help.)
I expect the list to include many plugins, but I couldn't think of much else in Vim's tree that might break. I'm happy to be wrong.
—
Reply to this email directly, view it on GitHub.
You are receiving this because you commented.![]()
—
Reply to this email directly, view it on GitHub.
You are receiving this because you are subscribed to this thread.![]()
I'm being deliberately vague about what they are to underscore the point that the submitter doesn't, nor has put forth any discernible effort to do so.
Sorry, that is not a convincing argument for reverting the change, and I am not going to take any request to revert such a change serious, when the submitte stays deliberately vague.
I can tell you why I considered merging this change of a new filetype:
The former method of using vim script variables seems like a hack, especially when both the bash and Posix shell scripts are more and more deviating. It does hurt maintaining runtime scripts and one always needs to consider if a change only affects a single filetype of several ones. Since we had the bash filetype already for a while, and it sources the existing shell runtime files, it did not feel like a big risk other than a differently named filetype.
—
Reply to this email directly, view it on GitHub.
You are receiving this because you commented.![]()
I second the comment RE: vagueness; while I understand and appreciate the position that the patch may not have accounted for all possible affects, deliberately withholding knowledge of defects will make it difficult to repair them!
I will continue to try to fix problems as I notice them, assuming for now we plan to keep the new filetype split. At the moment, I haven't come across any other than plugins, but I also am still on 9.1.0730 and haven't picked up this change yet.
—
Reply to this email directly, view it on GitHub.
You are receiving this because you commented.![]()
The previous behavior was a little confusing because it wasn't consistent... I was getting like 20% of the time the bash filetype and 80% the sh one. I get what you guys are saying but at the same time I believe that was really confusing. In my book either I did this or I would remove the bash filetype completely.
Having two names for a single filetype that still has to do a bunch of guesswork
Note that I will make some PRs to separate the logic between filetypes.
As Chris is saying bash and POSIX shell are diverging more and more it doesn't make sense to have bash logic mixed with sh logic.
—
Reply to this email directly, view it on GitHub.
You are receiving this because you commented.![]()
—
Reply to this email directly, view it on GitHub.
You are receiving this because you are subscribed to this thread.![]()
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.![]()
—
Reply to this email directly, view it on GitHub.
You are receiving this because you commented.![]()
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you commented.![]()
—
Reply to this email directly, view it on GitHub.
You are receiving this because you commented.![]()
Ok Rob, your point is much clearer and I think now we should rework better the sh filetype and use the compound filetype instead. I just don't understand why you are been such a cook about it. If you have some edge case that you like to be considered when changes. Just create some tests...
We all have different workflows and ways to do things. Even if I "did my homework" I probably wouldn't have the same problem that you are having simply because I don't use it like you are using it. Vim is built by the community and you help is needed we all work for free here and nobody has in mind every edge case when changing or developing. We all work for free and invest our own free time into this.
If you have issues on how thing are done speak up in a polite matter and come up with actual solutions without all the mining less digs to unpaid volunteers. The point you are making is valid but with your attitude is very difficult to actually taking you seriously...
—
Reply to this email directly, view it on GitHub.
You are receiving this because you commented.![]()
—
Reply to this email directly, view it on GitHub.
You are receiving this because you commented.![]()
—
Reply to this email directly, view it on GitHub.
You are receiving this because you commented.![]()
—
Reply to this email directly, view it on GitHub.
You are receiving this because you are subscribed to this thread.![]()
I thought this was going to be fixed. Or is what we have now the fix? I just updated to Vim 9.1.1029 and noticed that one of my plugins that tests for the filetype being "sh" fails because the filetype is now "bash". This also affects the nrrwrgn.vim plugin. Regards, Gary
I don't know if consensus was reached on reverting this (see also dense-analysis/ale#4888 where this is confusion on what is happening).
This seems like a classic example of Chesterton's Fence
—
Reply to this email directly, view it on GitHub.
You are receiving this because you commented.![]()
Alright, I'll revert it
—
Reply to this email directly, view it on GitHub.
You are receiving this because you commented.![]()
This seems like a classic example of Chesterton's Fence.
Yup.
—
Reply to this email directly, view it on GitHub.
You are receiving this because you commented.![]()
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you commented.![]()
This seems like a classic example of Chesterton's Fence.
May you explain that to me like I am 5? I am not sure I understand what you are tryin to say.
—
Reply to this email directly, view it on GitHub.
You are receiving this because you commented.![]()
This seems like a classic example of Chesterton's Fence.
May you explain that to me like I am 5? I am not sure I understand what you are tryin to say.
Quoting the linked article:
There exists […] let us say, for the sake of simplicity, a fence or gate erected across a road. The more modern type of reformer goes gaily up to it and says, "I don't see the use of this; let us clear it away." To which the more intelligent type of reformer will do well to answer: "If you don't see the use of it, I certainly won't let you clear it away. Go away and think. Then, when you can come back and tell me that you do see the use of it, I may allow you to destroy it."
In other words, only when all the tradeoffs of the change and when all the implications of the current situation are accounted for can we really say what's best.
"All" is intentionally hyperbolic: certainly not plausible for every single change. The reach of the change should inform the completeness of the due diligence, probably.
In other words, the more likely there is some impact, the more cautiously and informed we should probably tread.
(This particular change did not seem to account for "why is it like this" and "what else might be affected," so I imagine it felt hasty to myself and others. I hope we can all learn something about the nature of widely-used projects :) I certainly don't mean to imply anybody did anything wrong. Just that I might have made decisions differently, or that I wish other perspectives had weighed in.)
Feedback welcome; it has been difficult to put my sentiments into words around this in ways that are uplifting.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you commented.![]()
The problem is, it is not always entirely obvious how likely the impact of a change is. I still believe, it makes sense to handle bash as a separate filetype, after all, that's what a filetype is for. Yes I know bash is kind of a superset of sh, but that doesn't mean it shouldn't be a separate filetype.
However, I didn't expect this backlash and often we will notice such impacts only once a change is merged into master. It can sit around in the pull-request and we wouldn't notice the impact until is has been merged, no matter how-long we carefully think about a change. So it is hard to estimate if a change is causing trouble or not. I do usually think about such changes before merging, but my conclusion may not always be the same as yours or other users, so it's hard to make the right decision here.
In any case, I am trying to not cause any backwards incompatibility and while still allowing for some way to move on forward. But sometimes we may have to try things out and revert it if it causes issues. That's all what I can say to this.
—
Reply to this email directly, view it on GitHub.
You are receiving this because you commented.![]()
—
Reply to this email directly, view it on GitHub.
You are receiving this because you commented.![]()
I think you are skilled and experienced, but
the problem is your personal preference is stronger than (or you thought others personally against you) that actually was vim's classic mission or admitted truth
What does that even mean? Listen, I'm tired of you and your not really thought through arguments. Bram had personal preferences himself and a lot of what you are proclaiming as gospel is simply his personal preference.
I will reiterate WE ARE VOLUNTEERS. We work for free. Chris had to step up and start doing a lot more work and spend a lot of time, that maybe he would have spent in other ways, in managing changes and merging patches. Do you have an idea of the mental energy to do that? Do you understand that every stupid comments you make drains that finite energy? TBH if was Chris I would have ban you from the repo a long time ago but he so kind that wants to here you out Even though you are beening so freaking annoying.
To quote Chris in the other discussion #16368 (comment):
I am sorry, but you seem to really discourage working on and improving Vim. You are free to disagree but please don't try to cause frustration
I don't know where it has found the patience to respond to you like that but you are beening really annoying and really unreasonable to talk to.
So the lessons I would like you to learn are:
—
Reply to this email directly, view it on GitHub.
You are receiving this because you commented.![]()
—
Reply to this email directly, view it on GitHub.
You are receiving this because you commented.![]()
—
Reply to this email directly, view it on GitHub.
You are receiving this because you commented.![]()
—
Reply to this email directly, view it on GitHub.
You are receiving this because you commented.![]()
@chrisbra well, this has certainly gone in the opposite direction of what I hoped for in my reply (and thank you for your thoughtful reply). Perhaps we lock this thread for a while to give folks time to cool off ?
—
Reply to this email directly, view it on GitHub.
You are receiving this because you commented.![]()