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

test: Update cwd-enoent tests for AIX #2909

Closed
wants to merge 1 commit into from
Closed

test: Update cwd-enoent tests for AIX #2909

wants to merge 1 commit into from

Conversation

imran-iq
Copy link
Contributor

On AIX you can not remove a directory that you are currently inside of
as it results in an EBUSY error "EBUSY: resource busy or locked".
Updated the tests accordingly so that they are skipped on AIX.

On AIX you can not remove a directory that you are currently inside of
as it results in an EBUSY error "EBUSY: resource busy or locked".
Updated the tests accordingly so that they are skipped on AIX.
@mscdex mscdex added the test Issues and PRs related to the tests. label Sep 16, 2015
@cjihrig
Copy link
Contributor

cjihrig commented Sep 16, 2015

The code looks fine to me. @mhdawson LGTY?

@bnoordhuis
Copy link
Member

LGTM FWIW.

@mhdawson
Copy link
Member

LGTM

I asked Imran (iWuzHere ) to look at this as he is on the same team as me, and we've validated on our AIX boxes.

@mhdawson mhdawson self-assigned this Sep 16, 2015
@mhdawson
Copy link
Member

@bnoordhuis
Copy link
Member

@mhdawson Can you s/Update/update/ in the commit log before landing? Thanks.

@mhdawson
Copy link
Member

Some failures in CI run, but not related to what is being changed by this commit and see in previous CI runs, will land

mhdawson pushed a commit that referenced this pull request Sep 17, 2015
On AIX you can not remove a directory that you are currently inside of
as it results in an EBUSY error. "EBUSY: resource busy or locked".
Updated the tests accordingly so that they are skipped on AIX.

PR-URL: #2909
Reviewed-By: Ben Noordhuis <[email protected]>
Reviewed-By: Michael Dawson <[email protected]>
@mhdawson
Copy link
Member

landed as 1cfee8e

@Trott Trott closed this Sep 17, 2015
Fishrock123 pushed a commit that referenced this pull request Sep 20, 2015
On AIX you can not remove a directory that you are currently inside of
as it results in an EBUSY error. "EBUSY: resource busy or locked".
Updated the tests accordingly so that they are skipped on AIX.

PR-URL: #2909
Reviewed-By: Ben Noordhuis <[email protected]>
Reviewed-By: Michael Dawson <[email protected]>
@rvagg rvagg mentioned this pull request Sep 22, 2015
@MylesBorins
Copy link
Contributor

landed in lts-v4.x-staging as 3e09dcf

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
test Issues and PRs related to the tests.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants