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

Fix FreeBSD compatibilty on startup #34

Closed
wants to merge 1 commit into from

Conversation

@tony
Copy link
Contributor

tony commented Jan 11, 2016

Fixes this error in BSD-like systems:

    fd = os.open("/dev/tty", os.O_WRONLY)
    OSError: [Errno 6] Device not configured: '/dev/tty'

See also:

Fixes this error in BSD-like systems:
	fd = os.open("/dev/tty", os.O_WRONLY)
	OSError: [Errno 6] Device not configured: '/dev/tty'

See also:

- https://github.com/saltstack/salt/blob/6537da7/salt/utils/vt.py#L535
- Fixes #33
@eas604
Copy link
Contributor

eas604 commented Jan 12, 2016

Interesting! Is this check truly only needed for POSIX systems? Why wouldn't BSD need this check?

@jonathanslenders
Copy link
Member

jonathanslenders commented Jan 12, 2016

Thank you, @tony! Merged in: 464f456

Interesting indeed, ... I have no idea why we do this on Linux systems, while it doesn't work on BSD.

@tony tony deleted the tony:freebsd-forkfix branch May 9, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.