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

Added: SASL Compile options to postfix #4024

Merged
merged 3 commits into from Mar 5, 2018

Conversation

@Toasterson
Copy link
Contributor

Toasterson commented Mar 5, 2018

No description provided.

@Toasterson Toasterson requested review from xen0l and pyhalov Mar 5, 2018

AUXLIBS= -lssl
AUXLIBS+= -lcrypto
AUXLIBS+= -ldb-5.3
AUXLIBS+= -lsqlite3
AUXLIBS+= -L/usr/lib/sasl -lsasl

This comment has been minimized.

Copy link
@pyhalov

pyhalov Mar 5, 2018

Contributor

do we need corresponding -R flag?

This comment has been minimized.

Copy link
@Toasterson

Toasterson Mar 5, 2018

Author Contributor

Not as far as I know. CentOS/Fedora does not have a -R Flag. AFAIK many linuxes don't

This comment has been minimized.

Copy link
@pyhalov

pyhalov Mar 5, 2018

Contributor

Does it work without -L ?

This comment has been minimized.

Copy link
@Toasterson

Toasterson Mar 5, 2018

Author Contributor

Yep it works without -L. Don't know why the linuxes include it in that case good catch.

@@ -26,6 +26,10 @@ set name=org.opensolaris.consolidation value=$(CONSOLIDATION)

license $(COMPONENT_LICENSE_FILE) license='$(COMPONENT_LICENSE)'

<transform file -> add pkg.depend.bypass-generate libsasl\.so\.1>

depend fmri=system/library/security/libsasl type=require

This comment has been minimized.

Copy link
@pyhalov

pyhalov Mar 5, 2018

Contributor

why is it so complex?

This comment has been minimized.

Copy link
@Toasterson

Toasterson Mar 5, 2018

Author Contributor

No clue why but pkg seems not to like libsasl. It can not identify libsasl as a dependency thus it needs to be bypassed during the depend recognition run and then added manually. This is how I saw other packages solved that particular issue.

This comment has been minimized.

Copy link
@pyhalov

pyhalov Mar 5, 2018

Contributor

it seems pkgdepend finds dependency on libsasl fine here without this.

@pyhalov pyhalov merged commit a161241 into OpenIndiana:oi/hipster Mar 5, 2018
@Toasterson Toasterson deleted the Toasterson:postix-repair branch Mar 5, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.