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

Log also the output of PRE_BUILD command #333

Open
tenzap opened this issue Jun 4, 2023 · 2 comments
Open

Log also the output of PRE_BUILD command #333

tenzap opened this issue Jun 4, 2023 · 2 comments

Comments

@tenzap
Copy link

tenzap commented Jun 4, 2023

I have a call to a ./configure script in PRE_BUILD=

Is there a way to have the output logged? This could be useful in case the ./configure step fails.

If this is not possible it would be nice if it were logged.

@evelikov
Copy link
Collaborator

evelikov commented Jun 6, 2023

Currently only the build itself is logged, the rest - post_add, pre_build, post_build, pre_install, post_install and post_remove - are not.

Personally I don't see much value, especially since one of my dkms modules use any of those. That said, patches welcome - please ensure to handle all the pre/post entries listed above and update the documentation.

@anbe42
Copy link
Contributor

anbe42 commented Dec 6, 2023

output from applying patches should also be logged instead of going to the console

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants