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.

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

Missing and bad files in archive #132

Closed
remicollet opened this issue Sep 19, 2021 · 1 comment · Fixed by #135
Closed

Missing and bad files in archive #132

remicollet opened this issue Sep 19, 2021 · 1 comment · Fixed by #135
Assignees

Comments

@remicollet
Copy link

remicollet commented Sep 19, 2021

VERSION File is missing in archive so 1 test is failing

TEST 5/100 [tests/base/extension_info.phpt]
========DIFF========
001+ Warning: file_get_contents(/dev/shm/BUILD/php-zephir-parser-1.4.1/NTS/tests/base/../../VERSION): failed to open stream: No such file or directory in /dev/shm/BUILD/php-zephir-parser-1.4.1/NTS/tests/base/extension_info.php on line 10
002+ bool(false)
001- bool(true)
========DONE========
FAIL Test extension info [tests/base/extension_info.phpt] 

Noitice: using file outside the tests directory will be an issue, when installed in the pecl test tree.

Bad CR/LF

=====================================================================
FAILED TEST SUMMARY
---------------------------------------------------------------------
Tests recognizing wrapping C-code in CBLOCKs [tests/base/cblocks.phpt]
Comments before namespace [tests/comments/comments_before.phpt]
Syntax error when use unicode [tests/errors/bug30.phpt]
=====================================================================

Permissions

All files have executable perm, which is not needed / wanted.

Looks like this archive was generated under Windows.... terrible idea.... so is broken

@remicollet remicollet changed the title Bad file in archive. Missing and bad files in archive Sep 19, 2021
@Jeckerson
Copy link
Member

/cc @AlexNDRmac Lets use this issue for PECL package builder in CI.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants