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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

馃悰 A few related issues in upload commands #245

Open
nickmertin opened this issue Nov 11, 2022 · 0 comments 路 May be fixed by #261
Open

馃悰 A few related issues in upload commands #245

nickmertin opened this issue Nov 11, 2022 · 0 comments 路 May be fixed by #261
Assignees
Labels
bug Something isn't working

Comments

@nickmertin
Copy link

Expected Behavior:

Given a flat binary hello-world.bin, running the command pros upload --target v5 --slot 1 hello-world.bin from a directory which does not contain a project.pros file successfully upload the file to the V5. As described below, several variants on this command do not work.

Actual Behavior:

The following cases fail:

  • If a project.pros file exists in the current directory, then this error occurs: ERROR - pros.cli.upload:upload - No output files were found! Have you built your project?
  • If --slot 1 is omitted, then this error occurs: ERROR - pros.cli.main:main - unsupported operand type(s) for -=: 'NoneType' and 'int'
  • If we try to use ut instead of upload , we get this error (note that hello-world.bin is indeed provided on the command line): Error: Specify a file to upload or set the cwd inside a PROS project

Steps to reproduce:

Generate a flat binary (preferably a monolithic executable rather than a hot or cold package) from an existing PROS project, copy it out to a different directory, and try the above commands.

System information:

Operating System: Fedora 36

PROS Version: 3.3.3

Additional Information

vex-rt calls into the PROS CLI to upload, which is where we are getting this issue from.

Screenshots/Output Dumps/Stack Traces

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
None yet
Development

Successfully merging a pull request may close this issue.

3 participants