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

Which namespace to use for PECL packages in the PHP organization? #65

Closed
cmb69 opened this issue Oct 5, 2024 · 8 comments
Closed

Which namespace to use for PECL packages in the PHP organization? #65

cmb69 opened this issue Oct 5, 2024 · 8 comments
Labels
question Further information is requested

Comments

@cmb69
Copy link
Member

cmb69 commented Oct 5, 2024

This issue came up recently in php/pecl-file_formats-yaml#88 (review). Since it likely affects a lot of packages, I think it's worthwhile to track it here.

@asgrim
Copy link
Collaborator

asgrim commented Oct 5, 2024

Yeah no decision as such has been made, so maybe pecl is the best bet?

@asgrim
Copy link
Collaborator

asgrim commented Oct 7, 2024

@cmb69
Copy link
Member Author

cmb69 commented Oct 7, 2024

The timezonedb is already part of php-src; the PECL packages are just so users can update this, if they want to (works as drop-in replacement). So in my opinion, this can use the same namespace as other extensions.

@remicollet
Copy link
Member

IMHO and from Packaging PoV, I think pecl is the best choice and that php should be avoid.

In RPM (from Packaging Guidelines) we use php-<vendor>-<project>

So using pecl as namespace will be perfect, as will result in the same name (ex: php-pecl-apcu)

Switching to php will be very confusing, package will have to be

  • php-php-foo which looks terrible
  • php-foo (duplicated word removed), but this will be confusing wth ext build from "php-src"

@asgrim
Copy link
Collaborator

asgrim commented Oct 7, 2024

Worth noting that this specifically pertains to the "vendor namespace" part of the "package name" for the purposes of submitting the extension onto the Packagist site, not the "extension name" (which will remain the same as when it is/was on pecl.php.net).

So, for discussion sake (I am not saying this is the final decision), lets say we used the package name of pecl/timezonedb in composer.json, the extension name is still timezonedb as it always has been historically:

@asgrim
Copy link
Collaborator

asgrim commented Oct 9, 2024

No further comments, so I'll update the three affected repos to use pecl/ namespace.

@asgrim asgrim closed this as completed Oct 9, 2024
@asgrim asgrim modified the milestones: 0.1.0 - initial release, 0.1.0 Oct 10, 2024
@remicollet
Copy link
Member

Notice: use pecl as vendor name is not easy, see composer/packagist#1496

@asgrim
Copy link
Collaborator

asgrim commented Dec 13, 2024

Just for reference, some PHP Foundation discussion happened, the reference link is here. Apologies for the private link, but just wanted to "bookmark" it in a place relevant to the conversation.

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

No branches or pull requests

3 participants