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

Added stdio.rename #229

Closed
wants to merge 1 commit into from
Closed

Added stdio.rename #229

wants to merge 1 commit into from

Conversation

zevv
Copy link
Contributor

@zevv zevv commented Jun 29, 2015

added stdio.rename(). Even though rename() is offered by the lua os table it's probably nice to have it here as well for consistency.

(Sorry, still couldn't find the motivation to get the test suites to work so only the code, but no tests included.)

@gvvaughan
Copy link
Contributor

Hi Zevv!

Thanks, I'll add a test and commit presently.

No motivation required to run specl btw:

luarocks install specl
cd luaposix/luaposix
make check

And the specl yaml files are super easy too:

describe rename:
  # any lua code you like, indented a bit from the title

gvvaughan pushed a commit that referenced this pull request Jun 30, 2015
Closes #228, #229
* specs/posix_stdio_spec.yaml (rename): Specify behaviour of
posix.stdio.rename.
* ext/posix/stdio.c (Prename): Bind rename(2).
(posix_stdio_fns): Export Prename.

Signed-off-by: Gary V. Vaughan <gary@gnu.org>
@gvvaughan
Copy link
Contributor

merged at d9f5c65

@gvvaughan gvvaughan closed this Jun 30, 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.

None yet

2 participants