Drop warning about using the inn symlink #266
Conversation
Follow-up to just-merged #175. We no longer install the `inn` symlink, so users doing that manually should know what they are doing.
|
I would actually have held off on this because while end users that make their own links probably know what they are up to, but distro packagers (I'm deep into that world!) are sometimes a little less attentive and I think there are still distro packages that will add this link out of the box and might go on doing so for a while until somebody brings it to their attention. In any case this PR is obsolete since it was tacked onto #175, and I would treat it like water under the bridge. If any action is taken the best thing to do is probably run through the packaging that is out there (Repology is good for finding them) and directly submit bug reports to any that haven't removed the link. |
|
@alerque no problem with keeping this warning for a bit more time (releases). The intention was to keep the codebase clean from historic cruft (distro users shuold have been seeing fir warning for over a year).
Sorry not sure what you mean by this? |
|
I read the other PR wrong, it was a mention not a commit pushed to the branch. Disregard. As for timing you might be right, I forgot this has been in several releases already along with the documentation update, it was only the packaging generated here that finally got fixed. Sure go for it... |
|
I definitely see the usefulness in keeping a warning around, but since we're just checking I'd imagine our options are to remove it like this, or at least add a new |
|
Thanks everyone for the quick discussion iterations. I'll go ahead and merge this as-is. Let the |
Follow-up to just-merged #175. We no longer install the
innsymlink, so users doing that manually should know what they are doing.