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.

Sign up for GitHub

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

Jump to bottom

Windows: don't install __DECC_*.H #6303

Closed
wants to merge 1 commit into from
Closed

Windows: don't install __DECC_*.H #6303

wants to merge 1 commit into from

Conversation

levitte
Copy link
Member

@levitte levitte commented May 19, 2018

This adds the possibility to exclude files by regexp in util/copy.pl

Partial fix for #3254

@levitte
Windows: don't install __DECC_*.H
7f6bfa2 
This adds the possibility to exclude files by regexp in util/copy.pl

Partial fix for openssl#3254
@levitte levitte added branch: master Merge to master branch 1.1.0 branch: 1.1.1 Merge to OpenSSL_1_1_1-stable branch labels May 19, 2018
@levitte levitte mentioned this pull request May 19, 2018
Closed
9 tasks
@richsalz richsalz added the approval: done This pull request has the required number of approvals label May 19, 2018
levitte added a commit that referenced this pull request May 20, 2018
@levitte
Windows: don't install __DECC_*.H
246bd8f 
This adds the possibility to exclude files by regexp in util/copy.pl

Partial fix for #3254

Reviewed-by: Tim Hudson <tjh@openssl.org>
Reviewed-by: Rich Salz <rsalz@openssl.org>
(Merged from #6303)
levitte added a commit that referenced this pull request May 20, 2018
@levitte
Windows: don't install __DECC_*.H
12843c7 
This adds the possibility to exclude files by regexp in util/copy.pl

Partial fix for #3254

Reviewed-by: Tim Hudson <tjh@openssl.org>
Reviewed-by: Rich Salz <rsalz@openssl.org>
(Merged from #6303)

(cherry picked from commit 246bd8f)
@levitte
Copy link
Member Author

levitte commented May 20, 2018

Merged.

master / 1.1.1:
246bd8f Windows: don't install _DECC*.H

1.1.0:
12843c7 Windows: don't install _DECC*.H

@levitte levitte closed this May 20, 2018
@sav-ix
Copy link

sav-ix commented Jun 3, 2018

Hello, everyone,
Just tested recent master 6da34cf build using MSVC and found, that PR 12843c7 doesn't work (mean has no effect).

@levitte levitte mentioned this pull request Jun 5, 2018
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@richsalz richsalz richsalz approved these changes

@t-j-h t-j-h t-j-h approved these changes

Assignees
No one assigned
Labels
approval: done This pull request has the required number of approvals branch: master Merge to master branch branch: 1.1.1 Merge to OpenSSL_1_1_1-stable branch
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@levitte @sav-ix @richsalz @t-j-h