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

kgtk --progress cat Does Not Report Progress when Using a System Copy #678

Open
CraigMiloRogers opened this issue Oct 26, 2022 · 7 comments
Assignees
Labels
bug Something isn't working

Comments

@CraigMiloRogers
Copy link
Collaborator

kgtk --progress cat does not report progress when using a system copy.

@CraigMiloRogers CraigMiloRogers added the bug Something isn't working label Oct 26, 2022
@CraigMiloRogers CraigMiloRogers self-assigned this Oct 26, 2022
@CraigMiloRogers CraigMiloRogers added this to To do in KGTK Development via automation Oct 26, 2022
@CraigMiloRogers
Copy link
Collaborator Author

CraigMiloRogers commented Oct 26, 2022

bg=True should have been _bg=True to run the system command in the background.

@CraigMiloRogers
Copy link
Collaborator Author

The architecture for monitoring is wrong. The problem is that the subshell does not read the input file, aone or more cat/gunzip/etc. commands under the subshell are doing the input.

@CraigMiloRogers
Copy link
Collaborator Author

Apparently pv changes are causing problems.

@CraigMiloRogers
Copy link
Collaborator Author

pv was changed to not produce output when connected to a terminal but not on the foreground process group for that terminal.

@CraigMiloRogers
Copy link
Collaborator Author

CraigMiloRogers commented Oct 28, 2022

Three options:

  • Find a way to set pv's process group to the top process group
  • Pipe pv's output through cat
  • Implement a pv substitute in Python

@CraigMiloRogers
Copy link
Collaborator Author

Could not read the progress terminal's process group.

progress_startup: exception getting process monitoring tty control group for '/dev/pts/2': [Errno 25] Inappropriate ioctl for device

@CraigMiloRogers
Copy link
Collaborator Author

Piping pv's output through cat didn't work, either. Perhaps there's a problem in my current version of Linux?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Development

No branches or pull requests

1 participant