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

Strange results in Ubuntu #103

Open
yukiech opened this issue Sep 5, 2021 · 5 comments
Open

Strange results in Ubuntu #103

yukiech opened this issue Sep 5, 2021 · 5 comments

Comments

@yukiech
Copy link

yukiech commented Sep 5, 2021

The tester seems to "work" on Ubuntu with a "so:" rule with "-nostartfiles" but some result are strange.
maisaussi

I even tested to git clone other people libft and had about the same results.
Others testers compile thoses just fine and results are OK.

Anyone had this kind of issues ?

EDIT: A friend on an iMac tested my libft and thoses functions where OK.

@alexrowe99
Copy link

Have you got it working on Ubuntu?

@yukiech
Copy link
Author

yukiech commented Sep 14, 2021

Never managed to make thoses 3 work. (isalnum, toupper and tolower)
Looks like all the others are working tho.

(confirmed by sending my libft to someone on Mac who ran it for me, everything came back as OK, even isalnum/toupper/tolower)

@DCAndrei
Copy link

Hello, i am new to this, but i have the same issue.
On a mac with m1, ft_strrchr(test 7) and ft_memmove(test 9) give a crash, and ft_calloc a timeout on the first test, on Windows10 with ubutu wls, they are fine, but the ft_isalnum, ft_toupper and ft_tolower fail all. This is my experince, otherwise super tests. Thank you!

@yukiech
Copy link
Author

yukiech commented Nov 22, 2021

For ft_isalnum, ft_toupper and ft_tolower, the tester gives false errors. What i mean is: your function should pass the moulinette even if it says "Failed" on WSL2.
Overall, i can add that no libft tester can be 100% trusted. The moulinette does some additionnal tests. (Often, split, substr or strchr is OK on some testers but fails against the Moulinette)

probot-auto-merge bot added a commit that referenced this issue Sep 3, 2022
Pull request on the issue 103 (#103): an attempt to remove a strange behaviour of ft_isalnum, ft_tolower and ft_toupper functions in/on Ubuntu.
@github-actions
Copy link

github-actions bot commented Sep 3, 2022

Hello! Thanks for contributing to the libft unit test.

Note that this repository is not maintained by the owner anymore, instead there is a bot that will automatically merge any reviewed pull requests.
If you feel like it, here are some links that can help you submiting a change in the code base::

mofrim pushed a commit to mofrim/francinette that referenced this issue Jun 18, 2024
Pull request on the issue 103 (alelievr/libft-unit-test#103): an attempt to remove a strange behaviour of ft_isalnum, ft_tolower and ft_toupper functions in/on Ubuntu.
mofrim pushed a commit to mofrim/francinette that referenced this issue Jun 18, 2024
Pull request on the issue 103 (alelievr/libft-unit-test#103): an attempt to remove a strange behaviour of ft_isalnum, ft_tolower and ft_toupper functions in/on Ubuntu.
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

No branches or pull requests

3 participants