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

php-redis - re-publish pecl for php82 #17266

Open
wants to merge 1 commit into
base: oi/hipster
Choose a base branch
from

Conversation

fritzkink
Copy link
Contributor

for whatever reason disappeared from repository

@mtelka
Copy link
Contributor

mtelka commented May 21, 2024

@hoewweken
Copy link
Contributor

The jenkins log presents the error that i have seen locally:
https://jenkins.openindiana.aurora-opencloud.org/blue/organizations/jenkins/OpenIndiana%20Organisation%2Foi-userland/detail/PR-17266/1/pipeline

It doesn't build because there is a dublicate dir name/path here:

build/amd64/build/PHP-Parser-4.15.1/PHP-Parser-4.15.1/

@mtelka
Copy link
Contributor

mtelka commented May 21, 2024

Yes. IOW, it was never published, so it didn't disappear from the package repo :-).

@@ -20,6 +20,7 @@ include ../../../../make-rules/shared-macros.mk

COMPONENT_NAME= redis
COMPONENT_VERSION= 6.0.2
COMPONENT_REVISION= 1
COMPONENT_SRC= $(COMPONENT_NAME)-$(COMPONENT_VERSION)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There is no web/php-82/extension/php-redis yet so COMPONENT_REVISION is bogus here.

@AndWac
Copy link
Contributor

AndWac commented May 21, 2024

And this happens on the build server (and mine, too) and is the reason the package is in components.ignore:
x PHP-Parser-4.15.1/lib/PhpParser/PrettyPrinterAbstract.php, 61543 bytes, 121 tape blocks
PHP Warning: require(/jenkins/jobs/oi-userland/workspace/components/web/php/php-8_2-ext-redis/build/amd64/build/PHP-Parser-4.15.1/lib/PhpParser/Lexer/Emulative.php): Failed to open stream: No such file or directory in /jenkins/jobs/oi-userland/workspace/components/web/php/php-8_2-ext-redis/build/amd64/build/gen_stub.php on line 4653
PHP Fatal error: Uncaught Error: Failed opening required '/jenkins/jobs/oi-userland/workspace/components/web/php/php-8_2-ext-redis/build/amd64/build/PHP-Parser-4.15.1/lib/PhpParser/Lexer/Emulative.php' (include_path='.:/usr/php/8.2/share/PEAR') in /jenkins/jobs/oi-userland/workspace/components/web/php/php-8_2-ext-redis/build/amd64/build/gen_stub.php:4653
Stack trace:
#0 /jenkins/jobs/oi-userland/workspace/components/web/php/php-8_2-ext-redis/build/amd64/build/gen_stub.php(3751): {closure}('PhpParser\Lexer...')
#1 /jenkins/jobs/oi-userland/workspace/components/web/php/php-8_2-ext-redis/build/amd64/build/gen_stub.php(77): parseStubFile('<?php\n\n/**\n * @...')
#2 /jenkins/jobs/oi-userland/workspace/components/web/php/php-8_2-ext-redis/build/amd64/build/gen_stub.php(4697): processStubFile('/jenkins/jobs/o...', Object(Context))
#3 {main}
thrown in /jenkins/jobs/oi-userland/workspace/components/web/php/php-8_2-ext-redis/build/amd64/build/gen_stub.php on line 4653
make: *** [Makefile:196: /jenkins/jobs/oi-userland/workspace/components/web/php/php-8_2-ext-redis/build/amd64/redis_arginfo.h] Error 255
gmake: *** [/jenkins/jobs/oi-userland/workspace/make-rules/configure.mk:190: /jenkins/jobs/oi-userland/workspace/components/web/php/php-8_2-ext-redis/build/amd64/.installed] Error 2

@hoewweken
Copy link
Contributor

build/amd64/build/PHP-Parser-4.15.1/PHP-Parser-4.15.1/
I think it is reasonable to think that the dublicated dirname arrives from the extraction of the "v4.15.1.tar.gz".
The first directory name within the tar should be omitted somehow, or the target dir of the extract should be one directory above...
i don't yet see where this comes from, where php-parser-4.15.1.tar.gz comes linked into this stuff.. :-/

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

Successfully merging this pull request may close these issues.

4 participants