-
Notifications
You must be signed in to change notification settings - Fork 96
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
[FBW] Avoid duplicated donwloads, create networks and get hostname #506
Conversation
packages/first-boot-wizard/files/usr/lib/lua/firstbootwizard.lua
Outdated
Show resolved
Hide resolved
-- Start sharing lime-defaults and reboot | ||
share_defualts() | ||
remove_lock_file() | ||
os.execute("reboot 0") |
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.
what the 0 after reboot does?
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.
fix typo in 50582c1
packages/ubus-lime-fbw/Makefile
Outdated
@@ -25,6 +25,7 @@ define Package/$(PKG_NAME)/install | |||
$(CP) ./files/* $(1)/ | |||
@chmod a+x $(1)/usr/libexec/daemon/lime-fbw | |||
@chmod a+x $(1)/etc/init.d/ubus-lime-fbw | |||
@chmod a+x $(1)/www/cgi-bin/hostname |
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.
AFAIR some makefile macro should be used instead con plain chmod to install executables @spiccinini do you remember if it is like that?
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.
Yes, INSTALL_BIN
macro does chmod +x
(equivalent to chdmo a+x
)
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 package is using "the trick" using CP ./files/*
so using INSTALL_*
macros goes against that practicity. I don't know if there is a preferred way. I see mixed uses in other packages (from openwrt and libremesh).
packages/first-boot-wizard/files/usr/lib/lua/firstbootwizard/utils.lua
Outdated
Show resolved
Hide resolved
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.
see comments inline
This fixes #101 ? |
yes @spiccinini close #101 |
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.
Still one missing new line at EOF
Avoid duplicated downloads
Before it downloaded all the configurations from the associated devices each time it configured the network. Now it does it once per mac address.
Get remote hostname
Obtains the remote hostname the moment it gets the configurations and saves the data in the file.
Set username in create and join network functions
The user can input a hostname so that it changes the moment the new network configuration is set.