-
Notifications
You must be signed in to change notification settings - Fork 380
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
nodogsplash: explicit file copy #1027
Conversation
@BKPepe how is this? |
@BKPepe can I cherry-pick this into the other MRs for the other branches? Or do you want me to squash them? |
@BKPepe ping |
I will look into it tomorrow.
čt 19. 10. 2023 v 18:52 odesílatel Moritz Warning ***@***.***>
napsal:
… @BKPepe <https://github.com/BKPepe> ping
—
Reply to this email directly, view it on GitHub
<#1027 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AA7IDVC5PBJFSXNQUKXLZD3YAFLDRAVCNFSM6AAAAAA52Z4UI2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONZRGM3TCMRWGE>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
nodogsplash/Makefile
Outdated
$(CP) ./files/etc/config/nodogsplash $(1)/etc/config/ | ||
$(CP) ./files/etc/init.d/nodogsplash $(1)/etc/init.d/ | ||
$(CP) ./files/etc/uci-defaults/40_nodogsplash $(1)/etc/uci-defaults/ | ||
$(CP) ./files/usr/lib/nodogsplash/restart.sh $(1)/usr/lib/nodogsplash/ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This still does not look good. We are using simple CP. This needs to be reworked.
We should make it more clear, which will be more better to read this install section and it will be simple as it is.
- Create folder, then install files to it and then create another folder and install there files, which are there.
The order if you do first binaries, then config files, does not matter. Important is that it is not lost.
This is done for binary (a few lines above) and then you are creating three folders and it might be messy.
Anyway, I would suggest to use installation macros, which are listed here:
https://openwrt.org/docs/guide-developer/packages#file_installation_macros
E.g., INSTALL_CONF for config file, INSTALL_BIN for init config, etc.
This makes sure that it has permissions on the router, which we want and that it has not been tinkerered with.
6ad88ca
to
0e50a4d
Compare
Signed-off-by: Moritz Warning <moritzwarning@web.de>
Maintainer: me
Compile tested: pipeline