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

*: make CI green again #146

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

*: make CI green again #146

wants to merge 14 commits into from

Conversation

BusyJay
Copy link
Contributor

@BusyJay BusyJay commented Mar 10, 2020

This PR fixes several compilation warnings and skip function pointer for ctests.

I hope this can be a step to merging #130.

Signed-off-by: Jay Lee <BusyJayLee@gmail.com>
Signed-off-by: Jay Lee <BusyJayLee@gmail.com>
@BusyJay
Copy link
Contributor Author

BusyJay commented Mar 10, 2020

Allocation APIs have been changing rapidly since the end of January. I suggest to merge it first to make it build and test in stable channel and then fix alloc_trait feature when rapid refactor from wg-allocators finish.

BusyJay added 5 commits March 11, 2020 01:29
This reverts commit 18ee69f.

Signed-off-by: Jay Lee <BusyJayLee@gmail.com>
Signed-off-by: Jay Lee <BusyJayLee@gmail.com>
i686-apple-darwin is removed from nightly and beta.

Signed-off-by: Jay Lee <BusyJayLee@gmail.com>
Signed-off-by: Jay Lee <BusyJayLee@gmail.com>
Signed-off-by: Jay Lee <BusyJayLee@gmail.com>
Upstream doesn't guarantee configure file matches across platform and
versions. Checking configure file can fail easily.

Signed-off-by: Jay Lee <BusyJayLee@gmail.com>
@BusyJay BusyJay force-pushed the make-ci-green branch 3 times, most recently from ca62133 to 68f4419 Compare March 13, 2020 03:37
Signed-off-by: Jay Lee <BusyJayLee@gmail.com>
BusyJay added 2 commits March 14, 2020 03:05
Signed-off-by: Jay Lee <BusyJayLee@gmail.com>
Signed-off-by: Jay Lee <BusyJayLee@gmail.com>
@BusyJay
Copy link
Contributor Author

BusyJay commented Mar 23, 2020

@gnzlbg The only one left is the config diff. I don't know what's the purpose of the check and have no clue what's the proper way to fix it. Any suggestions?

@sfackler
Copy link
Collaborator

It checks that the version of the configure script that autoconf produces matches what's checked in here: /~https://github.com/gnzlbg/jemallocator/tree/master/jemalloc-sys/configure. You should just need to update that file.

@BusyJay
Copy link
Contributor Author

BusyJay commented Mar 23, 2020

But the check passes on MacOS but fail on Linux. I'm afraid updating it will just fix one and break the other.

BusyJay and others added 2 commits March 26, 2020 00:43
Signed-off-by: Jay Lee <busyjaylee@gmail.com>
Signed-off-by: Jay Lee <BusyJayLee@gmail.com>
@BusyJay
Copy link
Contributor Author

BusyJay commented Apr 22, 2020

@sfackler I followed your suggestion, and it did break MacOS build. I'm going to disable the check for now.

Signed-off-by: Jay Lee <BusyJayLee@gmail.com>
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.

2 participants