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

Next/50x/20201201/v2 #5625

Merged
merged 3 commits into from Dec 2, 2020

Conversation

victorjulien
Copy link
Member

Backport of #5614
#5623

PRScript output (if applicable):
Passed

jasonish and others added 3 commits December 1, 2020 10:52
Moving the libs specified in LDFLAGS to LDADD put them into the
correct placement on in the link command.

(cherry picked from commit 10f639e)
Prior to Rust 1.44, Cargo would name static libs with the .lib
extension. 1.44 changes this extension to .a when running under
a GNU environment on Windows like msys to make it more similar
to other unix environments.

Now assume static library name to be the same on Windows and
unix, but rename the .lib if found to still support older
versions of Rust on Windows.

(cherry picked from commit f3c59ef)
as they are in the protocol

cf bug OISF#2881

(cherry picked from commit 6b30890)
@victorjulien victorjulien requested review from jasonish, norg and a team as code owners December 2, 2020 12:29
@victorjulien victorjulien merged commit e309492 into OISF:master-5.0.x Dec 2, 2020
@victorjulien victorjulien deleted the next/50x/20201201/v2 branch December 7, 2020 07:51
regit added a commit to regit/suricata that referenced this pull request Oct 30, 2022
@regit regit mentioned this pull request Oct 30, 2022
3 tasks
benignbala pushed a commit to benignbala/suricata that referenced this pull request Nov 12, 2022
victorjulien pushed a commit to victorjulien/suricata that referenced this pull request Nov 13, 2022
Ticket: OISF#5625
(cherry picked from commit 4e1ff9a)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
3 participants