Skip to content

Commit

Permalink
fix: include stdout + stderr in error messages (#80)
Browse files Browse the repository at this point in the history
  • Loading branch information
mrcjkb authored Jul 28, 2023
1 parent 21fd8cc commit 5a8b967
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 1 deletion.
4 changes: 4 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,10 @@ All notable changes to this project will be documented in this file.
The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).

## [v5.1.3] - 2023-07-28
### Fixed
- Include stdout + stderr in error messages.

## [v5.1.2] - 2023-07-28
### Fixed
- Avoid duplicate `lua` dependencies in rockspec
Expand Down
2 changes: 1 addition & 1 deletion lua/luarocks-tag-release.lua
Original file line number Diff line number Diff line change
Expand Up @@ -76,7 +76,7 @@ local function luarocks_tag_release(package_name, package_version, specrev, args
local stdout = read_file(exec_out) or ''
local stderr = read_file(exec_err) or ''
if exit_code ~= 0 then
on_failure('FAILED (exit code ' .. exit_code .. '): ' .. cmd .. ' ' .. stderr)
on_failure(cmd .. ' FAILED\nexit code: ' .. exit_code .. '\nstdout: ' .. stdout .. '\nstderr: ' .. stderr)
end
return stdout, stderr
end
Expand Down

0 comments on commit 5a8b967

Please sign in to comment.