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

Code Improvements and Fixes #2

Open
wants to merge 9 commits into
base: master
Choose a base branch
from
Open

Code Improvements and Fixes #2

wants to merge 9 commits into from

Conversation

souravdas142
Copy link

Hello sir, I've seen some errors on your script while running Ubuntu 18.04.2 on both extract.sh and build.sh.

extract.sh : ./extract.sh: 12: ./extract.sh: cannot create ./scripts/gresource-extract/theme/icons/message-indicator-symbolic.svg: Directory nonexistent
you forgot to create a directory under theme named icons for this reason message-indicator-symbolic.svg: Directory nonexistent can't create. so I added a way to fix this issue.

build.sh : for previous issue, some fixes had been done for the build.sh and there was a dependency (libxml2-utils) issue creating a warining message (XMLLINT not set and xmllint not found in path; skipping xml preprocessing.) and i was not aware what may be effect happened for this warning. Thankfully I also manage to fix this issue by installing libxml2-utils

ALL ARE TESTED UNDER UBUNTU 18.04.2 LTS

I also aware by updating README.md. Please see.
please accept my pull request and merge with the master branch

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant