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

[Bug]: 1.21.1 /usr/share/doc/conky name mistake? #1928

Closed
0pLuS0 opened this issue May 20, 2024 · 2 comments · Fixed by #1934
Closed

[Bug]: 1.21.1 /usr/share/doc/conky name mistake? #1928

0pLuS0 opened this issue May 20, 2024 · 2 comments · Fixed by #1934
Labels
bug Bug report or bug fix PR triage Issue that hasn't been verified

Comments

@0pLuS0
Copy link

0pLuS0 commented May 20, 2024

What happened?

When I compile Conky from source in Linux 1.21.1 now has the /path named as; /usr/share/doc/conky-1.21.1-pre- that I copy the doc contents out of.

Previous versions were like this;

conky-1.19.6_pre
conky-1.20.1_pre

They use to always be as 'Version Number' _pre

Now in 1.21.1 (dash pre dash) -pre-

Which this seems a bit of odd naming.

Version

1.21.1

It would be nice in the next version updates the source compiled it as /usr/share/doc/conky-version

I never could understand all the _pre stuff...

THANKS

@0pLuS0 0pLuS0 added bug Bug report or bug fix PR triage Issue that hasn't been verified labels May 20, 2024
@Caellian Caellian added packaging Issue or PR related to conky packaging in general or for a specific distribution triage Issue that hasn't been verified and removed triage Issue that hasn't been verified packaging Issue or PR related to conky packaging in general or for a specific distribution labels May 20, 2024
@brndnmtthws
Copy link
Owner

How did you install it? Generally the -pre stuff should be removed from the versioning when -DRELEASE=ON with CMake, as shown here:

conky/cmake/Conky.cmake

Lines 179 to 184 in 7526e7e

if(RELEASE)
set(VERSION "${VERSION_MAJOR}.${VERSION_MINOR}.${VERSION_PATCH}")
else(RELEASE)
set(VERSION
"${VERSION_MAJOR}.${VERSION_MINOR}.${VERSION_PATCH}-pre-${GIT_SHORT_SHA}")
endif(RELEASE)

I'm trying to understand if this is a regression, or you just don't like the versioning on the docs dir.

Now that I think about it, it probably doesn't make sense to use the -pre and git sha in the version for the install path, so I might just remove that anyway.

brndnmtthws added a commit that referenced this issue May 23, 2024
We version the docs install path, but there's no reason to include the
-pre and git sha in the version for the path. With the git sha in
particular, it could result in having a bunch of dirs for each separate
git commit (annoying).

This resolves #1928.
brndnmtthws added a commit that referenced this issue May 23, 2024
We version the docs install path, but there's no reason to include the
-pre and git sha in the version for the path. With the git sha in
particular, it could result in having a bunch of dirs for each separate
git commit (annoying).

This resolves #1928.
@0pLuS0
Copy link
Author

0pLuS0 commented May 24, 2024

Hi @brndnmtthws

I am compiling Conky from source in Slackware, with a Slackware build script. I've actually been using this build script maybe for 2 years. DOCS are off; -DBUILD_DOCS=OFF

I have -DBUILD_DOCS=OFF because in Slack, you don't typically place docs under /usr/share but /usr/doc instead, that is why I am manually doing this and copying docs from the pre directory.

Unless i can turn it ON and all the docs are manually placed in /usr/share/docs with no need to copy from pre, and I just only need to make /usr/doc and then copy all the docs from /usr/share to /usr/doc?

I'm attaching the build script for you to see, it's actually called conky.SlackBuild

conky.SlackBuild.txt

THANKS

P.S.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Bug report or bug fix PR triage Issue that hasn't been verified
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants