-
Notifications
You must be signed in to change notification settings - Fork 4
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
Adapted package translation on the required amount of packages #39
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just the escaping needs to be improved...
src/modules/ProductCreator.rb
Outdated
Exec( | ||
Builtins.sformat( | ||
"rm %1/%2/%3/setup/descr/packages.*", | ||
String.Quote(basedir), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This will not work when the basedir
contains a space. There were quite some bug reports about this in the past, it seems users really use spaces in directory names... 😉
String.Quote is designed to be used inside single quotes, so this quoting will not work properly.
As we cannot use single quotes here I'd use Shellwords
here:
basedir.shellescape,
subdir.shellescape,
datadir.shellescape
(+ add require "shellwords"
ad the top).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Problem
While creating a new image the suse/setup/desc/packages.<language>.gz files will not be updated.
This produces an error while package installation. The package will not be found on the repo.
Solution
Deleting these corrupted files. So libzypp will not take it in account anymore.