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

design UI display unnormal when build a burnet package and install it firstly. #18

Closed
aglitke opened this issue Jul 2, 2013 · 3 comments
Assignees
Labels
Milestone

Comments

@aglitke
Copy link
Member

aglitke commented Jul 2, 2013

First time to make the burnet package. Setup.py just generates i18n files,
burnet.min.js and theme-default.min.css. But Setup.py does not not copy
them to the build directory.

@aglitke
Copy link
Member Author

aglitke commented Jul 2, 2013

Note that this can eventually be fixed properly by adopting an autotools based build system and properly specifying the build dependencies. For now I would approve a short-term fix that manually triggers the appropriate distutils build from within the package build process.

@shaohef
Copy link

shaohef commented Jul 3, 2013

This is too bad for the fresh user of Kimchi.
If he runs "python setup.py install" as the README.md says.
His kimchi can not works well.
Maybe he will give up Kimchi.

@aglitke
Copy link
Member Author

aglitke commented Jul 9, 2013

Fixed by: 0e2801e. Please reopen if you continue to encounter problems.

@aglitke aglitke closed this as completed Jul 9, 2013
patchew-importer pushed a commit to patchew-project/kimchi that referenced this issue Apr 23, 2018
This requirement started in a fixed Gingerbase bug (kimchi-project#18) that
reported lscpu problems when running in a French environment.
The output was being translated to french and breaking the
parsing made by the backend.

This is such a powder keg for all the plug-ins that I believe
this patch is justified. Instead of forcing en_US language
by using subprocess.Popen and setting the env, this is how
run_command will behave now:

- A new optional attribute, env_vars, was added. As the name
suggests, it allows the setup of the environment variables
to run the command.

- If env_vars is not set, run_command will copy the current
environment variables and set the language to the default
(en_US) by setting LC_ALL='C' variable. As of now, this is
the case of all the existing run_command calls in the code
for all WoK plug-ins. No behavior change will happen.

- If env_vars is set, but the LC_ALL var isn't, run_command
will set it to 'C' and force the default language.

- If env_vars is set and LC_ALL is also set, run_command
will not touch it and will run with env_vars as is. This
allows the caller to set the language at will.

Signed-off-by: Daniel Henrique Barboza <dhbarboza82@gmail.com>
patchew-importer pushed a commit to patchew-project/kimchi that referenced this issue May 25, 2018
This requirement started in a fixed Gingerbase bug (kimchi-project#18) that
reported lscpu problems when running in a French environment.
The output was being translated to french and breaking the
parsing made by the backend.

This is such a powder keg for all the plug-ins that I believe
this patch is justified. Instead of forcing en_US language
by using subprocess.Popen and setting the env, this is how
run_command will behave now:

- A new optional attribute, env_vars, was added. As the name
suggests, it allows the setup of the environment variables
to run the command.

- If env_vars is not set, run_command will copy the current
environment variables and set the language to the default
(en_US) by setting LC_ALL='C' variable. As of now, this is
the case of all the existing run_command calls in the code
for all WoK plug-ins. No behavior change will happen.

- If env_vars is set, but the LC_ALL var isn't, run_command
will set it to 'C' and force the default language.

- If env_vars is set and LC_ALL is also set, run_command
will not touch it and will run with env_vars as is. This
allows the caller to set the language at will.

Signed-off-by: Daniel Henrique Barboza <dhbarboza82@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants