Hi Jo,
On Tue, 24 Oct 2017, Jo-Philipp Wich wrote:
Hi,
comments inline.
Signed-off-by: Imre Kaloz <ka...@openwrt.org>
Given the fact that we explicitely wanted to avoid @openwrt.org mails
and that this was one of the discussion points leading to the split I
find it delicate to seal the merging commit with exactly such a S-o-b.
Combined with ...
$ git log --author "Imre Kaloz <ka...@openwrt.org>" --format=%cD -1
Tue, 18 Oct 2016 11:42:06 +0200
... this looks to me like nothing was learned from past experiences and
that all that matters here is having the own name attached to the work
being done.
LEDE does have patches in the tree s-o-b-ed and accepted like this dated
January '17. (f24ffb901e0408917748773b883841eca52eea05)
"*) email accounts
- currently there are around ~20 active openwrt.org mail accounts (the 3 owrt
devs would like to keep theirs active)
- turn all the webmaster@, hostmaster@, ... accounts into aliases that anyone
with voting rights can be subscribed to
- ask those people that are no longer active to voluntarily give up their
accounts
- mail addresses may under no conditions be used for any personal business,
consultancy, applying for jobs, ... purposes
- any mail sent from an openwrt.org account needs to adhere the trademark policy and
should only be used for FOSS purposes"
In my view, this is FOSS purpose rather than anything else.
Anyhow, on to the code.
diff --git a/Config.in b/Config.in
index ddbc4e2..d30c048 100644
--- a/Config.in
+++ b/Config.in
[...]
VERSION_REPO:=$(call qstrip_escape,$(CONFIG_VERSION_REPO))
-VERSION_REPO:=$(if
$(VERSION_REPO),$(VERSION_REPO),http://downloads.lede-project.org/snapshots)
+VERSION_REPO:=$(if
$(VERSION_REPO),$(VERSION_REPO),http://downloads.openwrt.org/snapshots)
We need to setup the server first or delegate downloads.openwrt.org to
downloads.lede-project.org or add a rewrite rule to redirect
http://downloads.openwrt.org/snapshots/(.*) to
http://downloads.lede-project.org/snapshots/$1 for now.
Agreed, there are dependencies on the infrastructure side for this to be
merged - or I'm OK to back it out until it's done. Having said that, this
is easier to crack compared to the $LEDE_GIT source entries in the
packages.
VERSION_BUG_URL:=$(call qstrip_escape,$(CONFIG_VERSION_BUG_URL))
-VERSION_BUG_URL:=$(if
$(VERSION_BUG_URL),$(VERSION_BUG_URL),http://bugs.lede-project.org/)
+VERSION_BUG_URL:=$(if
$(VERSION_BUG_URL),$(VERSION_BUG_URL),http://bugs.openwrt.org/)
Likewise.
Likewise.
VERSION_SUPPORT_URL:=$(call qstrip_escape,$(CONFIG_VERSION_SUPPORT_URL))
-VERSION_SUPPORT_URL:=$(if
$(VERSION_SUPPORT_URL),$(VERSION_SUPPORT_URL),http://forum.lede-project.org/)
+VERSION_SUPPORT_URL:=$(if
$(VERSION_SUPPORT_URL),$(VERSION_SUPPORT_URL),http://forum.openwrt.org/)
Undecided.
TBD.
VERSION_PRODUCT:=$(call qstrip_escape,$(CONFIG_VERSION_PRODUCT))
VERSION_PRODUCT:=$(if $(VERSION_PRODUCT),$(VERSION_PRODUCT),Generic)
[...]
diff --git a/package/base-files/files/etc/banner
b/package/base-files/files/etc/banner
index 4ec646a..af51b5a 100644
--- a/package/base-files/files/etc/banner
+++ b/package/base-files/files/etc/banner
@@ -1,10 +1,13 @@
- _________
- / /\ _ ___ ___ ___
- / LE / \ | | | __| \| __|
- / DE / \ | |__| _|| |) | _|
- /________/ LE \ |____|___|___/|___| lede-project.org
- \ \ DE /
- \ LE \ / -----------------------------------------------------------
- \ DE \ / %N (%V, %C)
- \________\/ -----------------------------------------------------------
-
+ _______ ________ __
+ | |.-----.-----.-----.| | | |.----.| |_
+ | - || _ | -__| || | | || _|| _|
+ |_______|| __|_____|__|__||________||__| |____|
+ |__| W I R E L E S S F R E E D O M
+ -----------------------------------------------------
+ DESIGNATED DRIVER (%C, %R)
Please replace "DESIGNATED DRIVER" with "%N". "%V, %C" (17.01.4,
r3560-79f57e422d) are more useful under the new version scheme than "%C,
%R" (r3560-79f57e422d, r3560-79f57e422d)
Let's stash the "DESIGNATED DRIVER" until others reply. VERSION_NUMBER
is currently SNAPSHOT - we should let others decide the cocktail part
of it. :)
+ -----------------------------------------------------
+ * 2 oz. Orange Juice Combine all juices in a
+ * 2 oz. Pineapple Juice tall glass filled with
+ * 2 oz. Grapefruit Juice ice, stir well.
+ * 2 oz. Cranberry Juice
+ -----------------------------------------------------
diff --git a/package/base-files/files/etc/init.d/system
b/package/base-files/files/etc/init.d/system
index 8c5f677..54c0e3e 100755
--- a/package/base-files/files/etc/init.d/system
+++ b/package/base-files/files/etc/init.d/system
@@ -7,7 +7,7 @@ USE_PROCD=1
validate_system_section()
{
uci_validate_section system system "${1}" \
- 'hostname:string:lede' \
+ 'hostname:string:openwrt' \
While we're at it we should properly capitalize the default to "OpenWrt".
Sure, this was to follow the "lede" capitalization, happy to change.
'conloglevel:uinteger' \
'buffersize:uinteger' \
'timezone:string:UTC' \
[...]
diff --git a/package/network/services/uhttpd/files/uhttpd.init
b/package/network/services/uhttpd/files/uhttpd.init
index 6b8be1c..fd8365a 100755
--- a/package/network/services/uhttpd/files/uhttpd.init
+++ b/package/network/services/uhttpd/files/uhttpd.init
@@ -52,7 +52,7 @@ generate_keys() {
[ -n "$GENKEY_CMD" ] && {
$GENKEY_CMD \
-days ${days:-730} -newkey rsa:${bits:-2048} -keyout
"${UHTTPD_KEY}.new" -out "${UHTTPD_CERT}.new" \
- -subj
/C="${country:-DE}"/ST="${state:-Saxony}"/L="${location:-Leipzig}"/O="${commonname:-Lede}$UNIQUEID"/CN="${commonname:-Lede}"
+ -subj
/C="${country:-DE}"/ST="${state:-Saxony}"/L="${location:-Leipzig}"/O="${commonname:-OpenWrt}$UNIQUEID"/CN="${commonname:-OpenWrt}"
Maybe we should adjust the defaults here to match the shipped config.
sync
mv "${UHTTPD_KEY}.new" "${UHTTPD_KEY}"
mv "${UHTTPD_CERT}.new" "${UHTTPD_CERT}"
diff --git a/package/network/utils/iproute2/Makefile
b/package/network/utils/iproute2/Makefile
index 14a33c5..c9ef626 100644
--- a/package/network/utils/iproute2/Makefile
+++ b/package/network/utils/iproute2/Makefile
@@ -108,7 +108,7 @@ define Build/Configure
$(PKG_BUILD_DIR)/Makefile
$(SED) "s,^LIBC_INCLUDE.*,LIBC_INCLUDE=$(STAGING_DIR)/include," \
$(PKG_BUILD_DIR)/Makefile
- echo "static const char SNAPSHOT[] =
\"$(PKG_VERSION)-$(PKG_RELEASE)-lede\";" \
+ echo "static const char SNAPSHOT[] =
\"$(PKG_VERSION)-$(PKG_RELEASE)-openwrt\";" \
> $(PKG_BUILD_DIR)/include/SNAPSHOT.h
endef
diff --git a/package/utils/busybox/Config-defaults.in
b/package/utils/busybox/Config-defaults.in
index 79059e5..2a8d9dd 100644
--- a/package/utils/busybox/Config-defaults.in
+++ b/package/utils/busybox/Config-defaults.in
@@ -2332,10 +2332,10 @@ config BUSYBOX_DEFAULT_FEATURE_NETSTAT_PRG
config BUSYBOX_DEFAULT_NSLOOKUP
bool
default n
-config BUSYBOX_DEFAULT_NSLOOKUP_LEDE
+config BUSYBOX_DEFAULT_NSLOOKUP_OPENWRT
Need to adjust the patch file accordingly.
Ack for all below, will do.
bool
default y
-config BUSYBOX_DEFAULT_FEATURE_NSLOOKUP_LEDE_LONG_OPTIONS
+config BUSYBOX_DEFAULT_FEATURE_NSLOOKUP_OPENWRT_LONG_OPTIONS
Dito.
bool
default n
config BUSYBOX_DEFAULT_NTPD
diff --git a/package/utils/busybox/config/networking/Config.in
b/package/utils/busybox/config/networking/Config.in
index 14875d5..2d2a1c4 100644
--- a/package/utils/busybox/config/networking/Config.in
+++ b/package/utils/busybox/config/networking/Config.in
@@ -700,17 +700,17 @@ config BUSYBOX_CONFIG_NSLOOKUP
default BUSYBOX_DEFAULT_NSLOOKUP
help
nslookup is a tool to query Internet name servers.
-config BUSYBOX_CONFIG_NSLOOKUP_LEDE
+config BUSYBOX_CONFIG_NSLOOKUP_OPENWRT
Dito.
bool "nslookup_lede"
depends on !BUSYBOX_CONFIG_NSLOOKUP
- default BUSYBOX_DEFAULT_NSLOOKUP_LEDE
+ default BUSYBOX_DEFAULT_NSLOOKUP_OPENWRT
Dito.
help
- nslookup is a tool to query Internet name servers (LEDE flavor).
+ nslookup is a tool to query Internet name servers (OpenWrt flavor).
-config BUSYBOX_CONFIG_FEATURE_NSLOOKUP_LEDE_LONG_OPTIONS
+config BUSYBOX_CONFIG_FEATURE_NSLOOKUP_OPENWRT_LONG_OPTIONS
Dito.
bool "Enable long options"
- default BUSYBOX_DEFAULT_FEATURE_NSLOOKUP_LEDE_LONG_OPTIONS
- depends on BUSYBOX_CONFIG_NSLOOKUP_LEDE && BUSYBOX_CONFIG_LONG_OPTS
+ default BUSYBOX_DEFAULT_FEATURE_NSLOOKUP_OPENWRT_LONG_OPTIONS
+ depends on BUSYBOX_CONFIG_NSLOOKUP_OPENWRT && BUSYBOX_CONFIG_LONG_OPTS
help
Support long options for the nslookup applet.
config BUSYBOX_CONFIG_NTPD
[...]
_______________________________________________
Lede-dev mailing list
Lede-dev@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/lede-dev
_______________________________________________
Lede-dev mailing list
Lede-dev@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/lede-dev
