| Commit message (Collapse) | Author | Age | Files | Lines |
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
* improve failure message to help user
I was trying to run `:Rocks install rocks.nvim` from neovim, which
results in:
```
luarocks "--force-lock" "--lua-version=5.1" "--tree=/home/teto/.local/share/nvim/rocks" "--server='https://luarocks.org/manifests/neorocks/'" "--server='https://lumen-oss.github.io/rocks-binaries/'" "install" "--force" "rocks.nvim"`
Installing https://luarocks.org/rocks.nvim-2.49.0-1.src.rock
Missing dependencies for rocks.nvim 2.49.0-1:
nvim-nio (not installed)
rtp.nvim (not installed)
rocks.nvim 2.49.0-1 depends on lua 5.1 (5.1-1 provided by VM: success)
rocks.nvim 2.49.0-1 depends on luarocks >= 3.11.1, < 4.0.0 (3.13.0-1 installed: success)
rocks.nvim 2.49.0-1 depends on toml-edit >= 0.6.1 (0.7.0-1 installed: success)
rocks.nvim 2.49.0-1 depends on fidget.nvim >= 1.1.0 (1.6.0-1 installed: success)
rocks.nvim 2.49.0-1 depends on fzy (1.0.3-1 installed: success)
rocks.nvim 2.49.0-1 depends on nvim-nio (not installed)
Installing https://luarocks.org/nvim-nio-1.10.1-1.src.rock
nvim-nio 1.10.1-1 is already installed in /home/teto/.local/share/nvim/rocks
Use --force to reinstall.
Error: Repository inconsistency detected (previously unfinished/corrupted installation?)
```
The error spaws from luarocks 3.13 so while the root problem of the
dependency not being detect while already installed might already have
been solved in newer releases. As rocks.nvim only displayed the last bit
(maybe because of stderr vs stdout distinction ?), patching the error message is still helpful.
* Update src/luarocks/deps.tl
Co-authored-by: Hisham Muhammad <hisham@gobolinux.org>
---------
Co-authored-by: Hisham Muhammad <hisham@gobolinux.org>
|
| |
|
|
|
| |
The variable `name` must be falsely in this `else` clause so stringify values like `false`, `nil `, `{}`, etc., so we can safely build an err message instead of merely crashing.
* #1828
* #1829
|
| |
|
|
| |
This fixes https://github.com/luarocks/luarocks/issues/1712.
|
| | |
|
| | |
|
| |
|
|
| |
rockspec_format argument.
|
| | |
|
| |
|
|
|
|
|
|
| |
When installing a rock that provides a module already provided by
another installed rock, print a warning indicating the collision
and which module will take priority.
Closes #1148
|
| |
|
|
|
|
| |
Same implementation as
https://github.com/leafo/moonrocks/commit/2e9585862497e65284bfb728542e5686ebec1280
by @leafo
|
| | |
|
| |
|
|
| |
Fixes #1874.
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
running
```luarocks config local_by_default``` returns ```Error: Unknown entry local_by_default```
and running
```luarocks config local_by_default true``` writes it as a string to the config file:
```lua
local_by_default = "true"
```
Same is true to any bool var set as false.
This pr aims to fix this issue.
|
| |
|
| |
Fixes #1827.
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
compat53 is vendored since #1757 as it is required to run luarocks with lua 5.1 or 5.2.
However, this introduced some issues as the GNUmakefile install rule places these in the same place where `luarocks install compat53` would install them. This means you get conflicts if you install the actual package:
```
Warning: /.../prefix/share/lua/5.1/compat53/init.lua is not tracked by this installation of LuaRocks. Moving it to /.../prefix/share/lua/5.1/compat53/init.lua~
Warning: /.../prefix/share/lua/5.1/compat53/module.lua is not tracked by this installation of LuaRocks. Moving it to /.../prefix/share/lua/5.1/compat53/module.lua~
Warning: /.../prefix/share/lua/5.1/compat53/file_mt.lua is not tracked by this installation of LuaRocks. Moving it to /.../prefix/share/lua/5.1/compat53/file_mt.lua~
```
It is also not ideal for linux package maintainers to include a vendored package, see: https://github.com/luarocks/luarocks/pull/1757#issuecomment-3409873412.
To solve these issues, this patchset makes the following changes:
- GNUmakefile now places the compat53 files under `luarocks/vendor/compat53` (which is added internally to the luarocks script's `package.path`). This way a user's installation of compat53 does not interfere at all with luarocks one.
- Added `--with-system-compat53` option to configure script for external packaging systems.
- Fixed install.bat's logic for deciding whether to vendor compat53, as the current script includes it for every version.
install.bat already places luarocks sources outside of LUAPATH, so that part can stay as is.
I've also inverted the version check to avoid the need for future patches like: #1850.
|
| | |
|
| |
|
|
| |
(#1825)
|
| | |
|
| | |
|
| |
|
|
| |
Co-authored-by: luau-project <luau.project@gmail.com>
|
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
| |
|
|
|
|
|
|
|
|
| |
Removing this broke the behavior of some luarocks.build
plugins. LuaRocks does not have a stable public API,
but since this function was returned as part of the
function signature, we'll revert this as a deprecated
function, to be removed in LuaRocks 4.0.
Closes #1800.
|
| | |
|
| | |
|
| | |
|
| |
|
|
|
|
|
| |
Rockspecs can access the directory of its dependencies using variables
in the format 'foo_ROCKDIR' where 'foo' is the name of a dependency.
This is used to be able to access files of the rock like 'conf', 'docs'
and more.
|
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Fixes #1755.
Changes:
* the idea applied as a fix was mirrored from the changes of #1616, by setting
```
defaults.fs_use_modules = false
```
in a suitable place for MSYS2.
* the line
```
if platforms.windows and not platforms.msys2_mingw_w64 and hardcoded.WIN_TOOLS then
```
is meant to follow a MSYS2 patch https://github.com/msys2/MINGW-packages/blob/343eeab4f185847981c6b644d775ee98717b6231/mingw-w64-lua-luarocks/0001-luarocks_msys2_mingw_w64.patch#L57 that fixes previous issues for them.
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
## Description
Fixes #1753.
Changes:
1. on each MSYS2 MinGW-w64 environment, there is a `cc.exe` that points to the default C compiler for the environment.
2. the line
```lua
defaults.variables.PWD = "cd"
```
is meant to follow a MSYS2 patch ( https://github.com/msys2/MINGW-packages/blob/5fd4997c2508f0949366513f0a2dd4347f9c4e77/mingw-w64-lua-luarocks/0001-luarocks_msys2_mingw_w64.patch#L48 ) that fixes previous issues for them.
|
| | |
|
| | |
|
| | |
|
| | |
|
| |
|
|
| |
See https://github.com/teal-language/tl/pull/890#issuecomment-2569388264
|
| |
|
|
| |
See https://github.com/teal-language/tl/pull/890#issuecomment-2569388264
|
| |
|
|
| |
Thanks to @euclidianAce's work at https://github.com/teal-language/tl/pull/869
|
| | |
|
| | |
|
| |
|
|
|
| |
We avoid doing a plain require() with a literal module name,
so that Cyan doesn't detect this as a circular dependency.
|
| |
|
|
| |
Break circular dependency detected by Cyan
|
| | |
|
| |
|
|
| |
Also addressed/removed all `--!` comments except for the ones that should stay.
|