Skip to content

Added a check to verify if the script path is absolute or not. #2204

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

Merged
merged 1 commit into from
May 20, 2025

Conversation

aaslam12
Copy link
Contributor

@aaslam12 aaslam12 commented May 19, 2025

Checklist

  • I have described the changes
  • I have linked to any relevant GitHub issues, if applicable
  • Documentation in doc/ has been updated
  • All new code is licensed under GPLv3

Description

#2192

Added a check to verify if the script path is absolute or not as it is buggy when you use '-c' cli argument to change conky.conf's file location to anywhere other than .config. View the issue for more information.

It also builds correctly.

Copy link

netlify bot commented May 19, 2025

Deploy Preview for conkyweb canceled.

Name Link
🔨 Latest commit 5ac3508
🔍 Latest deploy log https://app.netlify.com/projects/conkyweb/deploys/682b94ccb2a7040008c98b03

@github-actions github-actions bot added sources PR modifies project sources lua related to Lua integration in conky labels May 19, 2025
@brndnmtthws brndnmtthws merged commit d713933 into brndnmtthws:main May 20, 2025
29 checks passed
@brndnmtthws brndnmtthws added the bug related to incorrect existing implementation of some functionality label May 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug related to incorrect existing implementation of some functionality lua related to Lua integration in conky sources PR modifies project sources
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants