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

Fix to make the module a PostgreSQL extension. #6

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Fix to make the module a PostgreSQL extension. #6

wants to merge 3 commits into from

Conversation

snaga
Copy link

@snaga snaga commented Jul 11, 2015

Making the module an extension allows users to install just
only with 'CREATE EXTENSION data_encryption' instead of using
the install script (cipher_setup.sh).

And also it allows users to build with 'make USE_PGXS=1' instead
of using the dedicated build script.

This patch contains some fixes to allow building the module without
the PostgreSQL code tree to make it an extension.

snaga added 3 commits July 11, 2015 08:15
Making the module an extension allows users to install just
only with 'CREATE EXTENSION data_encryption' instead of using
the install script (cipher_setup.sh).

And also it allows users to build with 'make USE_PGXS=1' instead
of using the dedicated build script.
Now, tdeforpg can be installed as a PostgreSQL extension.
@t-kawabata-nec
Copy link
Contributor

Thank you for your proposal.

And I'm sorry for the delay replying.
(We were busy for readying 1st version release of tdeforpg EE).

I'm sorrier, could you wait for correspondence about a request a little more?

There are two reasons.

1st , We are considering Contributor License Agreement.
It is need some time to adjust our Legal Section.

2nd , we would like to examine how we maintain the consistency about Free Edition and Enterprise Edition installation.

@snaga
Copy link
Author

snaga commented Jul 25, 2015

Sure. No problem. Waiting for them.

@d-jung d-jung self-assigned this Aug 7, 2015
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.

3 participants