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 dynamic link path for envs in shell and add async poll in pipe #149

Merged
merged 3 commits into from
Aug 15, 2020

Conversation

yunwei37
Copy link
Contributor

@yunwei37 yunwei37 commented Aug 15, 2020

Now, if the command file is link to relative path, it can be run from shell directly.

  • if the command file is link to abstract path...I'm trying to figure it out in the rcore-fs.

other

  • add async_poll for pipe

@coveralls
Copy link

Pull Request Test Coverage Report for Build 209780179

  • 25 of 25 (100.0%) changed or added relevant lines in 4 files are covered.
  • 5 unchanged lines in 2 files lost coverage.
  • Overall coverage increased (+0.2%) to 46.578%

Files with Coverage Reduction New Missed Lines %
linux-object/src/fs/pipe.rs 1 87.18%
../../../../../usr/share/rust/.cargo/git/checkouts/rcore-fs-7fdf258332f6146d/517af47/rcore-fs/src/vfs.rs 4 8.51%
Totals Coverage Status
Change from base Build 208768481: 0.2%
Covered Lines: 6506
Relevant Lines: 13968

💛 - Coveralls

@yunwei37 yunwei37 marked this pull request as ready for review August 15, 2020 11:58
@wangrunji0408 wangrunji0408 merged commit a0e08f9 into rcore-os:master Aug 15, 2020
@wangrunji0408 wangrunji0408 added bug Something isn't working enhancement New feature or request labels Aug 30, 2020
zhangsn-19 pushed a commit to zhangsn-19/zCore that referenced this pull request Apr 23, 2022
fix dynamic link path for envs in shell and add async poll in pipe
@coveralls
Copy link

coveralls commented Jul 14, 2024

Pull Request Test Coverage Report for Build 209780179

Warning: This coverage report may be inaccurate.

This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

Details

  • 25 of 25 (100.0%) changed or added relevant lines in 4 files are covered.
  • 5 unchanged lines in 2 files lost coverage.
  • Overall coverage increased (+0.2%) to 46.578%

Files with Coverage Reduction New Missed Lines %
linux-object/src/fs/pipe.rs 1 87.18%
../../../../../usr/share/rust/.cargo/git/checkouts/rcore-fs-7fdf258332f6146d/517af47/rcore-fs/src/vfs.rs 4 8.51%
Totals Coverage Status
Change from base Build 208768481: 0.2%
Covered Lines: 6506
Relevant Lines: 13968

💛 - Coveralls

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants