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.

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

Wrap variable in double quotes #537

Merged
merged 1 commit into from
May 23, 2023
Merged

Wrap variable in double quotes #537

merged 1 commit into from
May 23, 2023

Conversation

Akasurde
Copy link
Member

  • avoid 'too many arguments' error when PKGMGR_OPTS contains spaces or special character

@Akasurde
Wrap variable in double quotes
b47a0eb 
* avoid 'too many arguments' error when PKGMGR_OPTS contains spaces
  or special character

Signed-off-by: Abhijeet Kasurde <akasurde@redhat.com>
@Akasurde Akasurde requested a review from a team as a code owner May 17, 2023 17:47
@Akasurde
Copy link
Member Author

Before -

+ '[' -z --nodocs --setopt=install_weak_deps=0 --setopt=rhel-8-for-x86_64-appstream-rpms.excludepkgs=ansible-core ']'
/output/scripts/assemble: line 46: [: too many arguments

@Akasurde Akasurde merged commit aabb04a into ansible:devel May 23, 2023
@Akasurde Akasurde deleted the too_many_arguments branch May 23, 2023 14:13
Shrews pushed a commit to Shrews/ansible-builder that referenced this pull request Jun 8, 2023
@Akasurde @Shrews
Wrap variable in double quotes (ansible#537)
b0084ba 
* avoid 'too many arguments' error when PKGMGR_OPTS contains spaces
  or special character

Signed-off-by: Abhijeet Kasurde <akasurde@redhat.com>
(cherry picked from commit aabb04a)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Shrews Shrews Shrews approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Akasurde @Shrews