这是indexloc提供的服务,不要输入任何密码
Skip to content

Conversation

@myypo
Copy link

@myypo myypo commented Dec 29, 2024

No description provided.


```sql
CREATE EXTENSION ulid;
CREATE EXTENSION pgx_ulid;
Copy link
Owner

Choose a reason for hiding this comment

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

Isn't this still the same as "ulid"?

Copy link
Author

Choose a reason for hiding this comment

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

Not in my case, at least.

postgres=# create extension ulid;
ERROR:  extension "ulid" is not available
DETAIL:  Could not open extension control file "/share/postgresql/extension/ulid.control": No such file or directory.
HINT:  The extension must first be installed on the system where PostgreSQL is running.
postgres=# create extension pgx_ulid;
CREATE EXTENSION

I am building pgx_ulid with the default Nix builder buildPgrxExtension here is the relevant builder's installation phase. And I don't see anything special/problematic about it, but it is the first time I am building something with pgrx, so I might be wrong!

Copy link
Owner

@pksunkara pksunkara Dec 31, 2024

Choose a reason for hiding this comment

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

Ah, that is an issue with nix packaging for this. As you can see in our release process, we rename the files (https://github.com/pksunkara/pgx_ulid/blob/master/.github/workflows/release.yml#L80). So, people using the debs would see the extension as ulid

Copy link
Author

Choose a reason for hiding this comment

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

Oh, oops, sorry, my silly mistake.

Neither mine derivation nor the one in the stalled PR to nixpkgs handles it correctly, since it was opened before the rename.
Might try helping get it merged later. Thank you.

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.

2 participants