-
Notifications
You must be signed in to change notification settings - Fork 2.3k
Rename files with a colon in the name #2465
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
Conversation
|
from #2463 (comment) ..
I hesitate to review/merge, since the colon is a very core concept of the methods that install templates, see option Lines 390 to 415 in 79c0245
This is just one point comes first into my mind .. may be there is more to do .. and it needs a lot of testing .. If we want touch this core methods we need good reasons for .. is to clone on windows such a good reason even when we will never support this platform? .. I don't think so, but this is my personal opinion. |
|
I'm for not restricting people from trying to use searxng on Windows, at the end of the end we both know that searxng being written in python will work on Windows. Thanks for the pointers about the other potential script breaking. Couldn't we move the install scripts to another repository? Similar to what we already have with https://github.com/searxng/searxng-docker |
On a (very) long term yes (may be) .. @dalf already asked for this multiple times in the past .. but this is a huge task .. we have a lot of shell scripts in the development- test- and deployment- cycles that make use of the By example most of the installation instructions are generated by theses scripts and my tests on distributions we support are based on this scripts. These changes will have far-reaching consequences, it will affect the CI, the way we develop today and it will affect all instances that have been built up to now. I have already started to modularize the shell scripts a bit to be able to remove parts of them gradually. I always have this in mind, but I don't see it as a task that can be done in one step ... the implications are too far reaching .. and I do not have alternatives at hand to generate documentation or test on various distributions .. I know there are a lot of test & build tools but they often need a infrastructure to setup first .. while these scripts are all running on a desktop or laptop .. These scripts can setup a build & test host in minutes https://docs.searxng.org/admin/buildhosts.html#buildhosts and with LXC the scripts scale over all distributions .. https://docs.searxng.org/dev/lxcdev.html Do you notice where we started: with one person who can't check out a clone on Windows and because of a colon we start to question everything .. I am alone with the development and would like to devote myself to more important issues that improve the product ... all these issues we discuss because of a colon do not bring SearXNG forward from a user point of view. To be honest, as long as there are no really good reasons, I would like to postpone the topic to the future. I mark this PR as draft for now.
Thats not true in deep .. SearXNG is a infrastructure deployed by docker container or installed by scripts. To name just two examples .. in the future redis becomes mandatory and there are hard coded concepts like the configuration expected under |
I strongly disagree, plenty of people are not deploying searxng with the install scripts or docker containers, and we shouldn't let down these peoples. Also I don't want to have redis as a hard requirement, we should offer features that benefit from having redis, but we should not make it a hard requirement. Running a simple searxng with just python should still be possible. Anyway it's probably going off topic and this should be discussed elsewhere. Anyway, on to the topic, like I said I don't want to restrict Windows users nor anyone running on another OS from trying to contribute to the project. If we want more contributors, we must not introduce too many barriers, or they will just never try the project. I'll one day try if these changes in this PR break existing installations. |
The issue is related to the underlying filesystem / these path names are not needed on MS-Win / |
|
Also:
I've lost hope that you change your mind on this subject. |
You are absolutely right, I would also like to remove all installation scripts from SearXNG in the long term. I have often made a start, but then there are always so many other tasks in between that are more urgent. I just want to save myself the extra work of touching the installation scripts again in the meantime. Even if it looks like a small thing, I have to do several tests (on all distros we support) ...
If it is explicitly desired, then we can reopen the issue and thie PR ... sadly I still don't have time for that. |
|
Don't have time for that anymore. |
What does this PR do?
This renames all the files that contain a colon in their name.
Why is this change important?
So that other OS like Windows can clone the repository: #2463
How to test this PR locally?
Author's checklist
Related issues
Closes #2463