Skip to content

Conversation

@Fangop
Copy link

@Fangop Fangop commented Jul 8, 2019

In readme, we know that when installing the header files should be installed in usr/local/include.
But, it does not do that in current version which leads to

gcc -o myprogram -lmimalloc myfile.c

failed.

So, I add a line in Cmakelists.txt to correct it!

mi_page_flags_t flags;
uint16_t capacity; // number of blocks committed
uint16_t reserved; // numbes of blocks reserved in memory
uint16_t reserved; // numbers of blocks reserved in memory
Copy link
Contributor

Choose a reason for hiding this comment

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

Don't mix typo fix here.

Copy link
Author

Choose a reason for hiding this comment

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

Ok, rebased commits.
Thanks!

@daanx
Copy link
Collaborator

daanx commented Jul 8, 2019

Ah thanks! However, I think the readme is wrong -- it should install to <installdir>/include, usually, /usr/local/lib/mimalloc-1.0/include which is what is already happening. When using the mimalloc from cmake this include path will be included automatically.

@daanx daanx closed this Jul 8, 2019
@Fangop
Copy link
Author

Fangop commented Jul 9, 2019

Hi @daanx !
So....., should I update the readme file to

install the library and header files in /usr/local/lib and /usr/local/lib/mimalloc-1.0/include

or (I preferred this one, since the 1.0 is not for good.)

install the library and header files in /usr/local/lib

or, just leave it to you? : )

daanx added a commit that referenced this pull request May 21, 2021
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.

3 participants