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.

Sign up for GitHub

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

Jump to bottom

Makefile changes #47

Merged
merged 4 commits into from
Jul 1, 2020
Merged

Makefile changes #47

merged 4 commits into from
Jul 1, 2020

Commits on Jun 23, 2020

  1. Makefile changed to fix a couple of issues. 

    The recursive approach used before is deperecated, new one reflects the
suggested behaviour ( see
https://www.gnu.org/software/make/manual/make.html#Phony-Targets ). Now
errors in sub-makes are tracked correctly.
    @Shub77
    Shub77 committed Jun 23, 2020
    Configuration menu
    Copy the full SHA
    03f8ad6 View commit details
    Browse the repository at this point in the history

  2. Missing double quotes

    @Shub77
    Shub77 committed Jun 23, 2020
    Configuration menu
    Copy the full SHA
    c5d1718 View commit details
    Browse the repository at this point in the history

  3. Added a check while installing 

    If make install is used more than once, an error will rise, since ln cannot
overwrite an already existing symbolic link. This is not a problem while
copying files in place, since cp -f will overwrite them anyway. Now the link
is not created if already there.
    @Shub77
    Shub77 committed Jun 23, 2020
    Configuration menu
    Copy the full SHA
    edec9bd View commit details
    Browse the repository at this point in the history

  4. Changed the dkms installing behaviour. 

    Now the copy operation to /usr/src of the needed files is performed by dkms.
This way, the module is recompiled by dkms immediately, and installed for
the running kernel. Since the modules does not need to be already compiled,
a make clean operation is performed before dkms-install/dkms-uninstall.
Since dkms will install the compiled modules in its dir tree, a make uninstall
operation is invoked too, so that only the modules compiled by dkms will remain
installed in the system. This happens on both dkms-install and dkms-uninstall.

The missing 'all' dependency was added to the install target.

The PWD variable was not appropriate, since was read by make from the shell
and never changed upon recursive make -C invocations. CURDIR variable is an
internal make variable, which is updated to reflect the real current working
directory.
    @Shub77
    Shub77 committed Jun 23, 2020
    Configuration menu
    Copy the full SHA
    106e370 View commit details
    Browse the repository at this point in the history