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

Fixed -Woverflow warning while compiling gcc-10 #246

Closed
wants to merge 1 commit into from
Closed

Fixed -Woverflow warning while compiling gcc-10 #246

wants to merge 1 commit into from

Conversation

Zaryob
Copy link

@Zaryob Zaryob commented Oct 11, 2020

In main commands source there is a -Woverflow gives me while making with make V=1 . After this patch that issue fixed.
Bests, regards.

in main command there is a -Woverflow error gives me. After making that it has
fixed.
@landley
Copy link
Owner

landley commented Oct 11, 2020

You have more than 2 billion commands built into toybox? How did you get a single ELF section that big?

I tried -Woverflow on gcc (x86-64 and i686) and clang, neither complained. What compiler version showed you this error?

@Zaryob
Copy link
Author

Zaryob commented Oct 11, 2020

You have more than 2 billion commands built into toybox? How did you get a single ELF section that big?

I tried -Woverflow on gcc (x86-64 and i686) and clang, neither complained. What compiler version showed you this error?

gcc --version
gcc (SUSE Linux) 10.2.1 20200825 [revision c0746a1beb1ba073c7981eb09f55b3d993b32e5c]
Copyright (C) 2020 Free Software Foundation, Inc.
This is free software; see the source for copying conditions.  There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.

@landley
Copy link
Owner

landley commented Oct 11, 2020

What is the actual error? How can an overflow happen here?

@Zaryob
Copy link
Author

Zaryob commented Oct 12, 2020

I searched that problems cause. It caused due to ld linker problem in openSUSE. I gonna fix my ld 😂 sorry for fuzzy pull request.

@Zaryob Zaryob closed this Oct 12, 2020
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.

None yet

2 participants