| Commit message (Collapse) | Author | Age | Files | Lines |
| |
|
|
| |
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.
|
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|