Skip to content

Building fixups#9012

Closed
levitte wants to merge 2 commits intoopenssl:masterfrom
levitte:fix-aix-20190526
Closed

Building fixups#9012
levitte wants to merge 2 commits intoopenssl:masterfrom
levitte:fix-aix-20190526

Conversation

@levitte
Copy link
Member

@levitte levitte commented May 27, 2019

AIX had the wrong flags for building modules, and a rm command had -v.

levitte added 2 commits May 27, 2019 14:20
Use -bnoentry, not -bexpall
Not all Unixen know the -v option
@levitte levitte added branch: master Applies to master branch branch: 1.1.1 Applies to OpenSSL_1_1_1-stable branch (EOL) labels May 27, 2019
Copy link
Member

@t8m t8m left a comment

Choose a reason for hiding this comment

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

LGTM

@t8m t8m added the approval: done This pull request has the required number of approvals label May 28, 2019
levitte added a commit that referenced this pull request May 28, 2019
Use -bnoentry, not -bexpall

Reviewed-by: Tomas Mraz <tmraz@fedoraproject.org>
(Merged from #9012)
levitte added a commit that referenced this pull request May 28, 2019
Not all Unixen know the -v option

Reviewed-by: Tomas Mraz <tmraz@fedoraproject.org>
(Merged from #9012)
levitte added a commit that referenced this pull request May 28, 2019
Use -bnoentry, not -bexpall

Reviewed-by: Tomas Mraz <tmraz@fedoraproject.org>
(Merged from #9012)

(cherry picked from commit f3d6a36)
levitte added a commit that referenced this pull request May 28, 2019
Not all Unixen know the -v option

Reviewed-by: Tomas Mraz <tmraz@fedoraproject.org>
(Merged from #9012)

(cherry picked from commit 3b8033f)
@levitte
Copy link
Member Author

levitte commented May 28, 2019

Merged.

master:
3b8033f Configurations/unix-Makefile.tmpl: not -v for rm
f3d6a36 Configure: link AIX modules correctly

1.1.1:
a043b21 Configurations/unix-Makefile.tmpl: not -v for rm
a52fa7a Configure: link AIX modules correctly

@levitte levitte closed this May 28, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approval: done This pull request has the required number of approvals branch: master Applies to master branch branch: 1.1.1 Applies to OpenSSL_1_1_1-stable branch (EOL)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants