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.

Sign up for GitHub

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

Jump to bottom

.init.lua isn't loaded under some circumstances #117

Closed
vladimir-kotikov opened this issue Sep 26, 2019 · 3 comments
Closed

.init.lua isn't loaded under some circumstances #117

vladimir-kotikov opened this issue Sep 26, 2019 · 3 comments
Labels
bug coming soon Work in progress

Comments

@vladimir-kotikov
Copy link
Owner

See cmderdev/cmder#957 and cmderdev/cmder#957 (comment) for background

Basically .init.lua is the root cause here as it modifies modules path so other modules can be loaded and I believe it just needs to be renamed into something else to not start with the dot (perhaps underscore should work, it just needs to be a symbol that goes before other ASCII symbols so that this module gets executed first by Clink)
@vladimir-kotikov
Copy link
Owner Author

Also cmderdev/cmder#2278

@chrisant996
Copy link
Collaborator

I'll rename the file. This should be coming in the next release, probably within the next 2 weeks.

@chrisant996 chrisant996 added bug coming soon Work in progress labels Aug 15, 2022
@chrisant996
Copy link
Collaborator

Closing because it is fixed in the dev branch, and closing it helps me keep track of the remaining work for the next release (which will include the dev changes).

chrisant996 added a commit that referenced this issue Aug 15, 2022
@chrisant996
Fix #117; .init.lua isn't loaded under some circumstances.
fd89185 
The "." at the beginning of the ".init.lua" file name marks it as a hidden file
in Linux and some sync programs.  That can cause sync programs to not copy the
file.  Renaming to "!init.lua" resolves that problem, while still ensuring the
script is loaded before any of the other completion scripts.
chrisant996 added a commit that referenced this issue Aug 15, 2022
@chrisant996
Fixing #117 requires a change in Cmder.
93c1b8f 
So, for best compatibility, clink-completions will have both .init.lua and
!init.lua files.
chrisant996 added a commit that referenced this issue Aug 20, 2022
@chrisant996
Dev - enhancements and fixes (#156)
fd6c9d1 
* WIP [common] Adds basic wrapper for tables

* WIP [common] Fixes tables wrapper to assign metatable correctly

* WIP [common] Rework matchers using new table wrapper

* Fix #103; show also remote branches in git merge.

* Fix #117; .init.lua isn't loaded under some circumstances.

The "." at the beginning of the ".init.lua" file name marks it as a hidden file
in Linux and some sync programs.  That can cause sync programs to not copy the
file.  Renaming to "!init.lua" resolves that problem, while still ensuring the
script is loaded before any of the other completion scripts.

* Fixing #117 requires a change in Cmder.

So, for best compatibility, clink-completions will have both .init.lua and
!init.lua files.

* Explicitly add the completions directory.

This enables newer Clink versions to load and use completions scripts on demand,
even in Cmder (which manually loads clink-completions scripts, so Clink doesn't
know about their script directory).

* Fix #16; add curl completions.

* Fix #145; add sudo argmatcher.

Requires Clink v1.3.40 to work fully properly.

* Rename to path_module so builtin path APIs are available.

* Fix base git flags bleeding thru to commands.

`git ls-files` doesn't have a linked argmatcher, so the `git` flags were
bleeding into it.  And etc for other commands.

* Smarter completions for git add and git checkout.

It's very unusual to add or checkout unchanged files.

So `git add` uses `git status -uall` to generate file matches.
And `git checkout` uses `git status -uno` to generate file matches.

* Fix `hide=true` for linked argmatchers.

* Fix #131; add winget completions.

* Simplify help_parser unsupported short circuit.

Return nothing, rather than returning an empty table.
This makes it easier for a caller to recognize it's not supported.

* matchers.ext_files improvements.

- It uses new APIs when available.
- It accepts a list of wildcards and adds them all at once, which makes
adding directories much more efficient (only one time).

* Add comments for some matchers module exports.

* Add some common argmatchers.

* Add robocopy argmatcher.

* Some explicit version checks, just in case.

* Add premake5 argmatcher.

* Add msbuild argmatcher.

* Add argmatchers for GNU grep and sed.

* Better capitalization function in robocopy argmatcher.

* Add adb/fastboot/scrcopy argmatchers.

* Fix typo in winget complete position.

* Fix #76; better display of yarn run completions.

This colors the yarn run matches.

In old Clink versions, it has no effect.

* Fix `yarn` completions.

While improving `yarn run` completions, `yarn` completions lost the
`run` matches.

Also, apply color to yarn local module matches, to differentiate them
from yarn commands.

* Clean luacheck warnings.

* Fix typo in yarn run completions.

* Fix some back-compat issues for v0.4.9.

* Tell luacheck about another Clink API.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug coming soon Work in progress
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@vladimir-kotikov @chrisant996