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

Cmake installs avrdude.1 in the wrong place on Linux #1064

Closed
stefanrueger opened this issue Aug 12, 2022 · 7 comments
Closed

Cmake installs avrdude.1 in the wrong place on Linux #1064

stefanrueger opened this issue Aug 12, 2022 · 7 comments
Labels
bug Something isn't working

Comments

@stefanrueger
Copy link
Collaborator

stefanrueger commented Aug 12, 2022

When I use cmake for installing avrdude.1 gets copied into the wrong place

$ ls -l /usr/local/share/man/
total 36K
drwxr-xr-x 2 root 4.0K Aug 12 16:03 man1
drwxrwxr-x 2 root  28K May 10 19:40 man3
drwxr-xr-x 2 root 4.0K Mar 17 22:37 man5
$ sudo cmake --build build_linux --target install
$ ls -l /usr/local/share/man/
total 84K
-rw-r--r-- 1 root  46K Aug  7 08:35 avrdude.1
drwxr-xr-x 2 root 4.0K Aug 12 16:03 man1
drwxrwxr-x 2 root  28K May 10 19:40 man3
drwxr-xr-x 2 root 4.0K Mar 17 22:37 man5

Using sudo make install in src copies avrdude.1 into the right place /usr/local/share/man/man1/avrdude.1 (on this system).

@stefanrueger stefanrueger added the bug Something isn't working label Aug 12, 2022
@sbngross
Copy link
Contributor

Might look into it if you like, since I brought this on to you and I'm quite familiar with cmake.

@mcuee
Copy link
Collaborator

mcuee commented Aug 12, 2022

Might look into it if you like, since I brought this on to you and I'm quite familiar with cmake.

That will be great. Pull requests are welcome. Thanks.

There are a few CMake related issues as well.

@sbngross
Copy link
Contributor

Well this one is easy enough. @stefanrueger should I put the fix on top of #1063 ?

@mcuee I can certainly try. But I'm note sure if I can provide for other OS's than linux.

@stefanrueger
Copy link
Collaborator Author

put the fix on top of PR #1063

Either way is fine. Thanks for looking into this!

@sbngross
Copy link
Contributor

sbngross commented Aug 12, 2022

Ran a short test with the cmake --build and the bare make in order to see if this breaks anything. As you noticed different behaviour of those two. Looks fine to me.

brutzzl3r@diamond-tiara ~/pkg % cmake --build tmp --target install          
[ 91%] Built target libavrdude
[100%] Built target avrdude
Install the project...
-- Install configuration: ""
...
-- Installing: /home/brutzzl3r/sandbox/opt/share/man/man1/avrdude.1

brutzzl3r@diamond-tiara ~/pkg % rm -rf ~/sandbox/opt              

brutzzl3r@diamond-tiara ~/pkg % make --no-print-directory -C tmp/src install
[ 91%] Built target libavrdude  
[100%] Built target avrdude
Install the project...
-- Install configuration: ""
...
-- Installing: /home/brutzzl3r/sandbox/opt/share/man/man1/avrdude.1

brutzzl3r@diamond-tiara ~/pkg % man -M /home/brutzzl3r/sandbox/opt/share/man -w 1 avrdude
/home/brutzzl3r/sandbox/opt/share/man/man1/avrdude.1

@mcuee
Copy link
Collaborator

mcuee commented Aug 13, 2022

@mcuee I can certainly try. But I'm note sure if I can provide for other OS's than linux.

I see that you are looking at #947 which is probably the most important of the issues related to CMake. Thanks.

#955 is half done if you can fix for Linux (macOS and FreeBSD will be similar).

#1035 seems to have a simple solution (option 1) even though Option 2 may be better.

#952 is diffcult and not only a CMake issue, rather it depends on other changes.

Pull request #960 was mostly working for Linux/macOS but not Windows. It was closed by the author.

@stefanrueger
Copy link
Collaborator Author

Fixed by PR #1063

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants