Skip to content

Make istty_stdout definition static#12

Open
ltning wants to merge 1 commit intokontaxis:masterfrom
ltning:patch-1
Open

Make istty_stdout definition static#12
ltning wants to merge 1 commit intokontaxis:masterfrom
ltning:patch-1

Conversation

@ltning
Copy link

@ltning ltning commented Feb 29, 2024

Without this, compilation fails with

/usr/local/bin/ld: /tmp//cc1rco4B.o:(.bss+0x2080220): multiple definition of `istty_stdout'; /tmp//ccIaYcw6.o:(.bss+0x0): first defined here
/usr/local/bin/ld: /tmp//cc1rco4B.o:(.bss+0x2080221): multiple definition of `istty_stderr'; /tmp//ccIaYcw6.o:(.bss+0x1): first defined here

..at least on FreeBSD and GCC 12. With this change, it compiles and runs as prescribed.

Without this, compilation fails with
```
/usr/local/bin/ld: /tmp//cc1rco4B.o:(.bss+0x2080220): multiple definition of `istty_stdout'; /tmp//ccIaYcw6.o:(.bss+0x0): first defined here
/usr/local/bin/ld: /tmp//cc1rco4B.o:(.bss+0x2080221): multiple definition of `istty_stderr'; /tmp//ccIaYcw6.o:(.bss+0x1): first defined here
```

..at least on FreeBSD and GCC 12. With this change, it compiles and runs as prescribed.
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.

1 participant