Skip to content

getopt doesn't handle missing spaces for single character options #2791

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.

Sign up for GitHub

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

Jump to bottom
Closed
jdm opened this issue Jul 3, 2012 · 0 comments
Closed

getopt doesn't handle missing spaces for single character options #2791

jdm opened this issue Jul 3, 2012 · 0 comments

Comments

@jdm
Copy link
Contributor

jdm commented Jul 3, 2012 

godot:build jdm$ rustc -L foo -Lbar
error: Argument to option 'L' missing.
godot:build jdm$ rustc -L foo -Lbar a
error: Unrecognized option: 'b'.

We should handle this properly.
@ghost ghost assigned jdm Jul 4, 2012
jdm added a commit to jdm/rust that referenced this issue Jul 4, 2012
@jdm
When getopts encounters an option that takes an argument, it should t…
135ffaf 
…reat the remainder of the current option string as the argument if it is non-empty. Fix rust-lang#2791.
@jdm jdm closed this as completed in 10fd195 Jul 4, 2012
@jdm jdm removed their assignment Jun 16, 2014
RalfJung pushed a commit to RalfJung/rust that referenced this issue Feb 26, 2023
@bors
Auto merge of rust-lang#2798 - LevitatingLion:master, r=oli-obk
ffd12f6 
Get Miri working on ARM

- Add a shim for `llvm.arm.hint`, which is required by `core::hint::spin_loop` on `arm` targets. The shim simply calls `yield_active_thread` on a YIELD hint, just like the shim for `llvm.aarch64.isb` that's already present.
- Change the signature of `miri_host_to_target_path` to use `c_char` instead of `i8`, to make it compatible with `CStr` on targets where `c_char` is unsigned. The implementation of `miri_host_to_target_path` accesses the memory as bytes and does not need to be adjusted.
- Enable ARM targets in CI. Specifically, `aarch64-unknown-linux-gnu` and `arm-unknown-linux-gnueabi` on the Linux host.

Since all tests also pass for `aarch64-unknown-linux-gnu` I took the liberty of adding that target to CI as well.

Fixes rust-lang#2791
celinval added a commit to celinval/rust-dev that referenced this issue Jun 4, 2024
@celinval @tautschnig @zhassan-aws
Update PR template (rust-lang#2791)
ed671fc 
Change the template to be more concise to address the team's concern that the template was too verbose.

The new template is also cleaner since we are now using the PR description as the base for the commit message.
---------
Co-authored-by: Michael Tautschnig <[email protected]>
Co-authored-by: Zyad Hassan <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@jdm