Skip to content
This repository has been archived by the owner on Dec 9, 2020. It is now read-only.

Fix several problems in install.sh #34

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Fix several problems in install.sh #34

wants to merge 1 commit into from

Conversation

SpotlightKid
Copy link

  • Fix shell syntax in 'xdg' and 'DIRECTORY' variable definitions.
  • Fix check whether theme file already exists to look at destination not source file.
  • Quote 'answer' shell variable.
  • Install themes with proper file permissions.
  • Make indentation consistent.

@SpotlightKid
Fix several problems in install.sh
4433d04 
* Fix shell syntax in 'xdg' and 'DIRECTORY' variable definitions.
* Fix check whether theme file already exists to look at
  destination not source file.
* Quote 'answer' shell variable.
* Install themes with proper file permissions.
* Make indentation consistent.

Signed-off-by: Christopher Arndt <chris@chrisarndt.de>
@SpotlightKid
Copy link
Author

The first problem is also addressed in #29, but this fixes additional problems.

@SpotlightKid
Copy link
Author

Ping...

@seamus-45 seamus-45 mentioned this pull request Oct 7, 2020
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
1 participant
@SpotlightKid