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

Workaround for ftell bug in Linux #533

Merged
merged 1 commit into from
Oct 21, 2015
Merged

Conversation

mkoval
Copy link
Collaborator

@mkoval mkoval commented Oct 20, 2015

ftell returns LONG_MAX as an error code, instead of -1L as specified in the documentation, when invoked on a directory. This behavior occurs on EXT4 filesystems, not but NFSv4 file systems, in kernel version 3.16.0 possibly with Ubuntu-specific patches applied (tested in Ubuntu 14.04).

This was not a fun one to debug. 😓

ftell returns LONG_MAX as an error code, instead of -1L as specified in
the documentation, when invoked on a directory. This behavior occurs on
EXT4 filesystems, not but NFSv4 file systems, in kernel version 3.16.0
possibly with Ubuntu-specific patches applied (tested in Ubuntu 14.04).
@jslee02
Copy link
Member

jslee02 commented Oct 20, 2015

Looks good to me 👍

System or compiler dependent behavior is mostly troublesome..

@mxgrey
Copy link
Member

mxgrey commented Oct 20, 2015

Looks good. 👍 AppVeyor might need a restart, though.

@jslee02 jslee02 added this to the DART 6.0.0 milestone Oct 20, 2015
@jslee02
Copy link
Member

jslee02 commented Oct 21, 2015

AppVeyor keeps failing for release mode tests because they exceeds the limited time, 60 min. But it should have nothing to do with this PR. Merging..

If anyone have an idea to boost AppVeyor buildings, I would like to share. 😓

jslee02 added a commit that referenced this pull request Oct 21, 2015
Workaround for ftell bug in Linux
@jslee02 jslee02 merged commit 37f7526 into dartsim:master Oct 21, 2015
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