[PATCH 0/8] bitbake.conf: set CONFFILES by default

[PATCH 0/8] bitbake.conf: set CONFFILES by default

[Robert Yang]

The following changes since commit 541315d6c56df6448f64c262f99d43d5c1e9400b:

  update_font_cache: only scan system font directories (2016-01-11 23:23:18 +0000)

are available in the git repository at:

  git://git.openembedded.org/openembedded-core-contrib rbt/conf
  http://cgit.openembedded.org/cgit.cgi/openembedded-core-contrib/log/?h=rbt/conf

Robert Yang (8):
  bitbake.conf: set CONFFILES by default
  recipes-bsp: cleanup CONFFILES
  recipes-connectivity: cleanup CONFFILES
  recipes-extended: cleanup CONFFILES
  recipes-core: cleanup CONFFILES
  recipes-devtools: cleanup CONFFILES
  recipes-graphics: cleanup CONFFILES
  recipes-multimedia/sato/support: cleanup CONFFILES

 meta/conf/bitbake.conf                             |    1 +
 meta/recipes-bsp/alsa-state/alsa-state.bb          |    1 -
 meta/recipes-bsp/pcmciautils/pcmciautils.inc       |    2 --
 meta/recipes-connectivity/avahi/avahi.inc          |    2 --
 meta/recipes-connectivity/bind/bind_9.10.3-P2.bb   |   11 -----------
 .../nfs-utils/nfs-utils_1.3.3.bb                   |    2 +-
 meta/recipes-connectivity/openssh/openssh_7.1p1.bb |    3 ---
 meta/recipes-connectivity/ppp/ppp_2.4.7.bb         |    1 -
 .../wpa-supplicant/wpa-supplicant_2.5.bb           |    1 -
 meta/recipes-core/base-files/base-files_3.0.14.bb  |    4 ----
 meta/recipes-core/busybox/busybox.inc              |    3 ---
 meta/recipes-core/dbus/dbus_1.8.20.bb              |    2 --
 meta/recipes-core/glibc/glibc-package.inc          |    1 -
 .../init-ifupdown/init-ifupdown_1.0.bb             |    1 -
 meta/recipes-core/initscripts/initscripts_1.0.bb   |    2 --
 meta/recipes-core/netbase/netbase_5.3.bb           |    2 --
 meta/recipes-core/readline/readline.inc            |    2 --
 meta/recipes-core/systemd/systemd_228.bb           |    7 -------
 .../sysvinit/sysvinit-inittab_2.88dsf.bb           |    1 -
 meta/recipes-core/udev/udev-extraconf_1.1.bb       |    1 -
 meta/recipes-devtools/opkg/opkg-arch-config_1.0.bb |    2 --
 meta/recipes-devtools/opkg/opkg_0.3.1.bb           |    1 -
 meta/recipes-extended/cronie/cronie_1.5.0.bb       |    1 -
 meta/recipes-extended/cups/cups.inc                |    1 -
 meta/recipes-extended/lighttpd/lighttpd_1.4.38.bb  |    2 --
 meta/recipes-extended/logrotate/logrotate_3.9.1.bb |    3 +--
 meta/recipes-extended/pam/libpam_1.2.1.bb          |    5 -----
 meta/recipes-extended/procps/procps_3.3.11.bb      |    2 --
 meta/recipes-extended/sudo/sudo.inc                |    2 --
 meta/recipes-extended/sysklogd/sysklogd.inc        |    1 -
 meta/recipes-extended/tzdata/tzdata_2015g.bb       |    3 ---
 meta/recipes-extended/xinetd/xinetd_2.3.15.bb      |    2 --
 .../xinput-calibrator/pointercal-xinput_0.0.bb     |    1 -
 .../xorg-xserver/xserver-xf86-config_0.1.bb        |    3 ---
 meta/recipes-multimedia/pulseaudio/pulseaudio.inc  |    8 --------
 .../pulseaudio-client-conf-sato_1.bb               |    1 -
 .../ca-certificates/ca-certificates_20150426.bb    |    4 ----
 37 files changed, 3 insertions(+), 89 deletions(-)

-- 
1.7.9.5

[PATCH 1/8] bitbake.conf: set CONFFILES by default

[Robert Yang]

There are a lot of binary packages have files in /etc/, but only a few
of them have set CONFFILES, more than 180 packages are not set in a
world build.  So treat all the files in /etc/ as CONFFILES as Debian
does:
- All the files in /etc/ are CONFFILES
- Move the file out of /etc/ if it is not a conffile
- If /etc/foo can't be moved out, and is changed during
  runtime, then create a symlink /etc/foo -> /var/foo

[YOCTO #8436]

Signed-off-by: Robert Yang <liezhi.yang@windriver.com>
---
 meta/conf/bitbake.conf |    1 +
 1 file changed, 1 insertion(+)

diff --git a/meta/conf/bitbake.conf b/meta/conf/bitbake.conf
index 2fe6b24..f0e4b3a 100644
--- a/meta/conf/bitbake.conf
+++ b/meta/conf/bitbake.conf
@@ -271,6 +271,7 @@ PACKAGE_BEFORE_PN ?= ""
 PACKAGES = "${PN}-dbg ${PN}-staticdev ${PN}-dev ${PN}-doc ${PN}-locale ${PACKAGE_BEFORE_PN} ${PN}"
 PACKAGES_DYNAMIC = "^${PN}-locale-.*"
 FILES = ""
+CONFFILES = "${sysconfdir}/*"
 
 FILES_${PN} = "${bindir}/* ${sbindir}/* ${libexecdir}/* ${libdir}/lib*${SOLIBS} \
             ${sysconfdir} ${sharedstatedir} ${localstatedir} \
-- 
1.7.9.5

[PATCH 2/8] recipes-bsp: cleanup CONFFILES

[Robert Yang]

The ${sysconfdir}/* are CONFFILES by default.

[YOCTO #8436]

Signed-off-by: Robert Yang <liezhi.yang@windriver.com>
---
 meta/recipes-bsp/alsa-state/alsa-state.bb    |    1 -
 meta/recipes-bsp/pcmciautils/pcmciautils.inc |    2 --
 2 files changed, 3 deletions(-)

diff --git a/meta/recipes-bsp/alsa-state/alsa-state.bb b/meta/recipes-bsp/alsa-state/alsa-state.bb
index d0f7bb3..ce59d33 100644
--- a/meta/recipes-bsp/alsa-state/alsa-state.bb
+++ b/meta/recipes-bsp/alsa-state/alsa-state.bb
@@ -52,7 +52,6 @@ RRECOMMENDS_alsa-state = "alsa-states"
 
 RDEPENDS_${PN} = "alsa-utils-alsactl"
 FILES_${PN} = "${sysconfdir}/init.d ${sysconfdir}/asound.conf"
-CONFFILES_${PN} = "${sysconfdir}/asound.conf"
 
 FILES_alsa-states = "${localstatedir}/lib/alsa/*.state"
 
diff --git a/meta/recipes-bsp/pcmciautils/pcmciautils.inc b/meta/recipes-bsp/pcmciautils/pcmciautils.inc
index 9d1c55b..0bf0620 100644
--- a/meta/recipes-bsp/pcmciautils/pcmciautils.inc
+++ b/meta/recipes-bsp/pcmciautils/pcmciautils.inc
@@ -28,5 +28,3 @@ EXTRA_OEMAKE = "-e 'STRIP=echo' 'LIB_OBJS=-lc -lsysfs' 'LEX=flex'"
 do_install () {
 	oe_runmake 'DESTDIR=${D}' install
 }
-
-CONFFILES_${PN} += "${sysconfdir}/pcmcia/config.opts"
-- 
1.7.9.5

[PATCH 3/8] recipes-connectivity: cleanup CONFFILES

[Robert Yang]

The ${sysconfdir}/* are CONFFILES by default.

[YOCTO #8436]

Signed-off-by: Robert Yang <liezhi.yang@windriver.com>
---
 meta/recipes-connectivity/avahi/avahi.inc          |    2 --
 meta/recipes-connectivity/bind/bind_9.10.3-P2.bb   |   11 -----------
 .../nfs-utils/nfs-utils_1.3.3.bb                   |    2 +-
 meta/recipes-connectivity/openssh/openssh_7.1p1.bb |    3 ---
 meta/recipes-connectivity/ppp/ppp_2.4.7.bb         |    1 -
 .../wpa-supplicant/wpa-supplicant_2.5.bb           |    1 -
 6 files changed, 1 insertion(+), 19 deletions(-)

diff --git a/meta/recipes-connectivity/avahi/avahi.inc b/meta/recipes-connectivity/avahi/avahi.inc
index 825197d..966dfe0 100644
--- a/meta/recipes-connectivity/avahi/avahi.inc
+++ b/meta/recipes-connectivity/avahi/avahi.inc
@@ -113,8 +113,6 @@ RRECOMMENDS_avahi-dev_append_libc-glibc = " gettext-dev"
 
 RRECOMMENDS_avahi-dev[nodeprrecs] = "1"
 
-CONFFILES_avahi-daemon = "${sysconfdir}/avahi/avahi-daemon.conf"
-
 INITSCRIPT_PACKAGES = "avahi-daemon avahi-dnsconfd"
 INITSCRIPT_NAME_avahi-daemon = "avahi-daemon"
 INITSCRIPT_PARAMS_avahi-daemon = "defaults 21 19"
diff --git a/meta/recipes-connectivity/bind/bind_9.10.3-P2.bb b/meta/recipes-connectivity/bind/bind_9.10.3-P2.bb
index 875a0c8..06165b1 100644
--- a/meta/recipes-connectivity/bind/bind_9.10.3-P2.bb
+++ b/meta/recipes-connectivity/bind/bind_9.10.3-P2.bb
@@ -93,14 +93,3 @@ do_install_append() {
 	install -m 0644 ${WORKDIR}/bind9 ${D}${sysconfdir}/default
 }
 
-CONFFILES_${PN} = " \
-	${sysconfdir}/bind/named.conf \
-	${sysconfdir}/bind/named.conf.local \
-	${sysconfdir}/bind/named.conf.options \
-	${sysconfdir}/bind/db.0 \
-	${sysconfdir}/bind/db.127 \
-	${sysconfdir}/bind/db.empty \
-	${sysconfdir}/bind/db.local \
-	${sysconfdir}/bind/db.root \
-	"
-
diff --git a/meta/recipes-connectivity/nfs-utils/nfs-utils_1.3.3.bb b/meta/recipes-connectivity/nfs-utils/nfs-utils_1.3.3.bb
index 2e0c8b1..30f068d 100644
--- a/meta/recipes-connectivity/nfs-utils/nfs-utils_1.3.3.bb
+++ b/meta/recipes-connectivity/nfs-utils/nfs-utils_1.3.3.bb
@@ -75,7 +75,7 @@ CONFFILES_${PN}-client += "${localstatedir}/lib/nfs/etab \
 			   ${localstatedir}/lib/nfs/rmtab \
 			   ${localstatedir}/lib/nfs/xtab \
 			   ${localstatedir}/lib/nfs/statd/state \
-			   ${sysconfdir}/nfsmount.conf"
+			   "
 
 FILES_${PN}-client = "${base_sbindir}/*mount.nfs* ${sbindir}/*statd \
 		      ${sbindir}/rpc.idmapd ${sbindir}/sm-notify \
diff --git a/meta/recipes-connectivity/openssh/openssh_7.1p1.bb b/meta/recipes-connectivity/openssh/openssh_7.1p1.bb
index d81127c..cf0bd71 100644
--- a/meta/recipes-connectivity/openssh/openssh_7.1p1.bb
+++ b/meta/recipes-connectivity/openssh/openssh_7.1p1.bb
@@ -149,9 +149,6 @@ RCONFLICTS_${PN} = "dropbear"
 RCONFLICTS_${PN}-sshd = "dropbear"
 RCONFLICTS_${PN}-keygen = "ssh-keygen"
 
-CONFFILES_${PN}-sshd = "${sysconfdir}/ssh/sshd_config"
-CONFFILES_${PN}-ssh = "${sysconfdir}/ssh/ssh_config"
-
 ALTERNATIVE_PRIORITY = "90"
 ALTERNATIVE_${PN}-scp = "scp"
 ALTERNATIVE_${PN}-ssh = "ssh"
diff --git a/meta/recipes-connectivity/ppp/ppp_2.4.7.bb b/meta/recipes-connectivity/ppp/ppp_2.4.7.bb
index 5f06034..0a1063e 100644
--- a/meta/recipes-connectivity/ppp/ppp_2.4.7.bb
+++ b/meta/recipes-connectivity/ppp/ppp_2.4.7.bb
@@ -75,7 +75,6 @@ do_install_append () {
 	chmod u+s ${D}${sbindir}/pppd
 }
 
-CONFFILES_${PN} = "${sysconfdir}/ppp/pap-secrets ${sysconfdir}/ppp/chap-secrets ${sysconfdir}/ppp/options"
 PACKAGES =+ "${PN}-oa ${PN}-oe ${PN}-radius ${PN}-winbind ${PN}-minconn ${PN}-password ${PN}-l2tp ${PN}-tools"
 FILES_${PN}        = "${sysconfdir} ${bindir} ${sbindir}/chat ${sbindir}/pppd ${systemd_unitdir}/system/ppp@.service"
 FILES_${PN}-oa       = "${libdir}/pppd/${PV}/pppoatm.so"
diff --git a/meta/recipes-connectivity/wpa-supplicant/wpa-supplicant_2.5.bb b/meta/recipes-connectivity/wpa-supplicant/wpa-supplicant_2.5.bb
index 2126377..111429a 100644
--- a/meta/recipes-connectivity/wpa-supplicant/wpa-supplicant_2.5.bb
+++ b/meta/recipes-connectivity/wpa-supplicant/wpa-supplicant_2.5.bb
@@ -34,7 +34,6 @@ PACKAGES_prepend = "wpa-supplicant-passphrase wpa-supplicant-cli "
 FILES_wpa-supplicant-passphrase = "${bindir}/wpa_passphrase"
 FILES_wpa-supplicant-cli = "${sbindir}/wpa_cli"
 FILES_${PN} += "${datadir}/dbus-1/system-services/*"
-CONFFILES_${PN} += "${sysconfdir}/wpa_supplicant.conf"
 
 do_configure () {
 	${MAKE} -C wpa_supplicant clean
-- 
1.7.9.5

[PATCH 4/8] recipes-extended: cleanup CONFFILES

[Robert Yang]

The ${sysconfdir}/* are CONFFILES by default.

[YOCTO #8436]

Signed-off-by: Robert Yang <liezhi.yang@windriver.com>
---
 meta/recipes-extended/cronie/cronie_1.5.0.bb       |    1 -
 meta/recipes-extended/cups/cups.inc                |    1 -
 meta/recipes-extended/lighttpd/lighttpd_1.4.38.bb  |    2 --
 meta/recipes-extended/logrotate/logrotate_3.9.1.bb |    3 +--
 meta/recipes-extended/pam/libpam_1.2.1.bb          |    5 -----
 meta/recipes-extended/procps/procps_3.3.11.bb      |    2 --
 meta/recipes-extended/sudo/sudo.inc                |    2 --
 meta/recipes-extended/sysklogd/sysklogd.inc        |    1 -
 meta/recipes-extended/tzdata/tzdata_2015g.bb       |    3 ---
 meta/recipes-extended/xinetd/xinetd_2.3.15.bb      |    2 --
 10 files changed, 1 insertion(+), 21 deletions(-)

diff --git a/meta/recipes-extended/cronie/cronie_1.5.0.bb b/meta/recipes-extended/cronie/cronie_1.5.0.bb
index 697501a..922ba00 100644
--- a/meta/recipes-extended/cronie/cronie_1.5.0.bb
+++ b/meta/recipes-extended/cronie/cronie_1.5.0.bb
@@ -78,4 +78,3 @@ do_install_append () {
 }
 
 FILES_${PN} += "${sysconfdir}/cron*"
-CONFFILES_${PN} += "${sysconfdir}/crontab"
diff --git a/meta/recipes-extended/cups/cups.inc b/meta/recipes-extended/cups/cups.inc
index 5aa365d..bc8f19a 100644
--- a/meta/recipes-extended/cups/cups.inc
+++ b/meta/recipes-extended/cups/cups.inc
@@ -116,7 +116,6 @@ FILES_${PN} += "${datadir}/doc/cups/images \
                 ${datadir}/doc/cups/*.css \
                 ${datadir}/icons/ \
                "
-CONFFILES_${PN} += "${sysconfdir}/cups/cupsd.conf"
 
 SYSROOT_PREPROCESS_FUNCS += "cups_sysroot_preprocess"
 cups_sysroot_preprocess () {
diff --git a/meta/recipes-extended/lighttpd/lighttpd_1.4.38.bb b/meta/recipes-extended/lighttpd/lighttpd_1.4.38.bb
index 7cfc141..6f6a14e 100644
--- a/meta/recipes-extended/lighttpd/lighttpd_1.4.38.bb
+++ b/meta/recipes-extended/lighttpd/lighttpd_1.4.38.bb
@@ -67,8 +67,6 @@ do_install_append() {
 
 FILES_${PN} += "${sysconfdir} /www"
 
-CONFFILES_${PN} = "${sysconfdir}/lighttpd.conf"
-
 PACKAGES_DYNAMIC += "^lighttpd-module-.*"
 
 python populate_packages_prepend () {
diff --git a/meta/recipes-extended/logrotate/logrotate_3.9.1.bb b/meta/recipes-extended/logrotate/logrotate_3.9.1.bb
index 5f1a601..6688d8e 100644
--- a/meta/recipes-extended/logrotate/logrotate_3.9.1.bb
+++ b/meta/recipes-extended/logrotate/logrotate_3.9.1.bb
@@ -27,8 +27,7 @@ PACKAGECONFIG ?= "\
 PACKAGECONFIG[acl] = ",,acl"
 PACKAGECONFIG[selinux] = ",,libselinux"
 
-CONFFILES_${PN} += "${localstatedir}/lib/logrotate.status \
-		    ${sysconfdir}/logrotate.conf"
+CONFFILES_${PN} += "${localstatedir}/lib/logrotate.status"
 
 # If RPM_OPT_FLAGS is unset, it adds -g itself rather than obeying our
 # optimization variables, so use it rather than EXTRA_CFLAGS.
diff --git a/meta/recipes-extended/pam/libpam_1.2.1.bb b/meta/recipes-extended/pam/libpam_1.2.1.bb
index 3e69f11..cc5cb0f 100644
--- a/meta/recipes-extended/pam/libpam_1.2.1.bb
+++ b/meta/recipes-extended/pam/libpam_1.2.1.bb
@@ -163,8 +163,3 @@ addtask pam_sanity before do_configure
 
 BBCLASSEXTEND = "nativesdk native"
 
-CONFFILES_${PN}-runtime += "${sysconfdir}/pam.d/common-session"
-CONFFILES_${PN}-runtime += "${sysconfdir}/pam.d/common-auth"
-CONFFILES_${PN}-runtime += "${sysconfdir}/pam.d/common-password"
-CONFFILES_${PN}-runtime += "${sysconfdir}/pam.d/common-session-noninteractive"
-CONFFILES_${PN}-runtime += "${sysconfdir}/pam.d/common-account"
diff --git a/meta/recipes-extended/procps/procps_3.3.11.bb b/meta/recipes-extended/procps/procps_3.3.11.bb
index c6dccc6..e842e47 100644
--- a/meta/recipes-extended/procps/procps_3.3.11.bb
+++ b/meta/recipes-extended/procps/procps_3.3.11.bb
@@ -42,8 +42,6 @@ do_install_append () {
         fi
 }
 
-CONFFILES_${PN} = "${sysconfdir}/sysctl.conf"
-
 bindir_progs = "free pkill pmap pgrep pwdx skill snice top uptime"
 base_bindir_progs += "kill pidof ps watch"
 base_sbindir_progs += "sysctl"
diff --git a/meta/recipes-extended/sudo/sudo.inc b/meta/recipes-extended/sudo/sudo.inc
index a25876b..2862450 100644
--- a/meta/recipes-extended/sudo/sudo.inc
+++ b/meta/recipes-extended/sudo/sudo.inc
@@ -19,8 +19,6 @@ inherit autotools
 PACKAGECONFIG ??= ""
 PACKAGECONFIG[zlib] = "--enable-zlib,--disable-zlib,zlib"
 
-CONFFILES_${PN} = "${sysconfdir}/sudoers"
-
 EXTRA_OECONF = "--with-editor=/bin/vi --with-env-editor"
 
 EXTRA_OECONF_append_libc-musl = " --disable-hardening "
diff --git a/meta/recipes-extended/sysklogd/sysklogd.inc b/meta/recipes-extended/sysklogd/sysklogd.inc
index 85b3cdc..aed4412 100644
--- a/meta/recipes-extended/sysklogd/sysklogd.inc
+++ b/meta/recipes-extended/sysklogd/sysklogd.inc
@@ -22,7 +22,6 @@ SRC_URI = "http://www.infodrom.org/projects/sysklogd/download/sysklogd-${PV}.tar
 SRC_URI_append_e500v2 = " file://no-vectorization.patch"
 
 INITSCRIPT_NAME = "syslog"
-CONFFILES_${PN} = "${sysconfdir}/syslog.conf.${BPN}"
 
 CFLAGS_append = " -DSYSV"
 
diff --git a/meta/recipes-extended/tzdata/tzdata_2015g.bb b/meta/recipes-extended/tzdata/tzdata_2015g.bb
index a6faa1d..4a573d1 100644
--- a/meta/recipes-extended/tzdata/tzdata_2015g.bb
+++ b/meta/recipes-extended/tzdata/tzdata_2015g.bb
@@ -207,6 +207,3 @@ FILES_${PN} += "${datadir}/zoneinfo/Pacific/Honolulu     \
                 ${datadir}/zoneinfo/zone.tab             \
                 ${datadir}/zoneinfo/iso3166.tab          \
                 ${datadir}/zoneinfo/Etc/*"
-
-CONFFILES_${PN} += "${@ "${sysconfdir}/timezone" if bb.utils.to_boolean(d.getVar('INSTALL_TIMEZONE_FILE', True)) else "" }"
-CONFFILES_${PN} += "${sysconfdir}/localtime"
diff --git a/meta/recipes-extended/xinetd/xinetd_2.3.15.bb b/meta/recipes-extended/xinetd/xinetd_2.3.15.bb
index 652fe91..3996bfa 100644
--- a/meta/recipes-extended/xinetd/xinetd_2.3.15.bb
+++ b/meta/recipes-extended/xinetd/xinetd_2.3.15.bb
@@ -68,5 +68,3 @@ do_install() {
 	       -e 's,@SBINDIR@,${sbindir},g' \
 	       ${D}${systemd_unitdir}/system/xinetd.service
 }
-
-CONFFILES_${PN} = "${sysconfdir}/xinetd.conf"
-- 
1.7.9.5

[PATCH 5/8] recipes-core: cleanup CONFFILES

[Robert Yang]

The ${sysconfdir}/* are CONFFILES by default.

[YOCTO #8436]

Signed-off-by: Robert Yang <liezhi.yang@windriver.com>
---
 meta/recipes-core/base-files/base-files_3.0.14.bb  |    4 ----
 meta/recipes-core/busybox/busybox.inc              |    3 ---
 meta/recipes-core/dbus/dbus_1.8.20.bb              |    2 --
 meta/recipes-core/glibc/glibc-package.inc          |    1 -
 .../init-ifupdown/init-ifupdown_1.0.bb             |    1 -
 meta/recipes-core/initscripts/initscripts_1.0.bb   |    2 --
 meta/recipes-core/netbase/netbase_5.3.bb           |    2 --
 meta/recipes-core/readline/readline.inc            |    2 --
 meta/recipes-core/systemd/systemd_228.bb           |    7 -------
 .../sysvinit/sysvinit-inittab_2.88dsf.bb           |    1 -
 meta/recipes-core/udev/udev-extraconf_1.1.bb       |    1 -
 11 files changed, 26 deletions(-)

diff --git a/meta/recipes-core/base-files/base-files_3.0.14.bb b/meta/recipes-core/base-files/base-files_3.0.14.bb
index d391707..d61b58b 100644
--- a/meta/recipes-core/base-files/base-files_3.0.14.bb
+++ b/meta/recipes-core/base-files/base-files_3.0.14.bb
@@ -174,7 +174,3 @@ FILES_${PN} = "/"
 FILES_${PN}-doc = "${docdir} ${datadir}/common-licenses"
 
 PACKAGE_ARCH = "${MACHINE_ARCH}"
-
-CONFFILES_${PN} = "${sysconfdir}/fstab ${@['', '${sysconfdir}/hostname'][(d.getVar('hostname', True) != '')]} ${sysconfdir}/shells"
-CONFFILES_${PN} += "${sysconfdir}/motd ${sysconfdir}/nsswitch.conf ${sysconfdir}/profile"
-
diff --git a/meta/recipes-core/busybox/busybox.inc b/meta/recipes-core/busybox/busybox.inc
index 9541123..858468a 100644
--- a/meta/recipes-core/busybox/busybox.inc
+++ b/meta/recipes-core/busybox/busybox.inc
@@ -42,9 +42,6 @@ INITSCRIPT_NAME_${PN}-udhcpd = "busybox-udhcpd"
 SYSTEMD_PACKAGES = "${PN}-syslog"
 SYSTEMD_SERVICE_${PN}-syslog = "busybox-syslog.service"
 
-CONFFILES_${PN}-syslog = "${sysconfdir}/syslog-startup.conf.${BPN}"
-CONFFILES_${PN}-mdev = "${sysconfdir}/mdev.conf"
-
 RRECOMMENDS_${PN} = "${PN}-syslog ${PN}-udhcpc"
 
 inherit cml1 systemd update-rc.d ptest
diff --git a/meta/recipes-core/dbus/dbus_1.8.20.bb b/meta/recipes-core/dbus/dbus_1.8.20.bb
index 0c7f6b1..9063273 100644
--- a/meta/recipes-core/dbus/dbus_1.8.20.bb
+++ b/meta/recipes-core/dbus/dbus_1.8.20.bb
@@ -39,8 +39,6 @@ USERADD_PARAM_${PN} = "--system --home ${localstatedir}/lib/dbus \
                        --no-create-home --shell /bin/false \
                        --user-group messagebus"
 
-CONFFILES_${PN} = "${sysconfdir}/dbus-1/system.conf ${sysconfdir}/dbus-1/session.conf"
-
 DEBIANNAME_${PN} = "dbus-1"
 
 PACKAGES =+ "${PN}-lib"
diff --git a/meta/recipes-core/glibc/glibc-package.inc b/meta/recipes-core/glibc/glibc-package.inc
index bad6424..c7d44af 100644
--- a/meta/recipes-core/glibc/glibc-package.inc
+++ b/meta/recipes-core/glibc/glibc-package.inc
@@ -208,7 +208,6 @@ pkg_postinst_nscd () {
 		fi
 	fi
 }
-CONFFILES_nscd="${sysconfdir}/nscd.conf"
 
 SYSTEMD_PACKAGES = "nscd"
 SYSTEMD_SERVICE_nscd = "nscd.service"
diff --git a/meta/recipes-core/init-ifupdown/init-ifupdown_1.0.bb b/meta/recipes-core/init-ifupdown/init-ifupdown_1.0.bb
index 733ae41..e8d87be 100644
--- a/meta/recipes-core/init-ifupdown/init-ifupdown_1.0.bb
+++ b/meta/recipes-core/init-ifupdown/init-ifupdown_1.0.bb
@@ -38,4 +38,3 @@ PACKAGE_ARCH_qemuall = "${MACHINE_ARCH}"
 RDEPENDS_${PN} = "netbase"
 RCONFLICTS_${PN} = "netbase (< 1:5.0)"
 
-CONFFILES_${PN} = "${sysconfdir}/network/interfaces"
diff --git a/meta/recipes-core/initscripts/initscripts_1.0.bb b/meta/recipes-core/initscripts/initscripts_1.0.bb
index f90de6e..bd3795f 100644
--- a/meta/recipes-core/initscripts/initscripts_1.0.bb
+++ b/meta/recipes-core/initscripts/initscripts_1.0.bb
@@ -178,5 +178,3 @@ pkg_postinst_${PN} () {
         rm -f "/etc/volatile.cache"
     fi
 }
-
-CONFFILES_${PN} += "${sysconfdir}/init.d/checkroot.sh"
diff --git a/meta/recipes-core/netbase/netbase_5.3.bb b/meta/recipes-core/netbase/netbase_5.3.bb
index ccd89ff..7cff754 100644
--- a/meta/recipes-core/netbase/netbase_5.3.bb
+++ b/meta/recipes-core/netbase/netbase_5.3.bb
@@ -20,5 +20,3 @@ do_install () {
 	install -m 0644 etc-protocols ${D}${sysconfdir}/protocols
 	install -m 0644 etc-services ${D}${sysconfdir}/services
 }
-
-CONFFILES_${PN} = "${sysconfdir}/hosts"
diff --git a/meta/recipes-core/readline/readline.inc b/meta/recipes-core/readline/readline.inc
index 35df8b4..f4fd077 100644
--- a/meta/recipes-core/readline/readline.inc
+++ b/meta/recipes-core/readline/readline.inc
@@ -38,5 +38,3 @@ do_install_append () {
 }
 
 BBCLASSEXTEND = "native nativesdk"
-
-CONFFILES_${PN} += "${sysconfdir}/inputrc"
diff --git a/meta/recipes-core/systemd/systemd_228.bb b/meta/recipes-core/systemd/systemd_228.bb
index aa2e846..cbad8b8 100644
--- a/meta/recipes-core/systemd/systemd_228.bb
+++ b/meta/recipes-core/systemd/systemd_228.bb
@@ -280,13 +280,6 @@ RRECOMMENDS_${PN}-binfmt = "kernel-module-binfmt-misc"
 
 RRECOMMENDS_${PN}-vconsole-setup = "kbd kbd-consolefonts kbd-keymaps"
 
-CONFFILES_${PN} = "${sysconfdir}/machine-id \
-                ${sysconfdir}/systemd/coredump.conf \
-                ${sysconfdir}/systemd/journald.conf \
-                ${sysconfdir}/systemd/logind.conf \
-                ${sysconfdir}/systemd/system.conf \
-                ${sysconfdir}/systemd/user.conf"
-
 FILES_${PN} = " ${base_bindir}/* \
                 ${datadir}/bash-completion \
                 ${datadir}/dbus-1/services \
diff --git a/meta/recipes-core/sysvinit/sysvinit-inittab_2.88dsf.bb b/meta/recipes-core/sysvinit/sysvinit-inittab_2.88dsf.bb
index f539da8d..7a8b2ea 100644
--- a/meta/recipes-core/sysvinit/sysvinit-inittab_2.88dsf.bb
+++ b/meta/recipes-core/sysvinit/sysvinit-inittab_2.88dsf.bb
@@ -77,7 +77,6 @@ fi
 PACKAGE_ARCH = "${MACHINE_ARCH}"
 
 FILES_${PN} = "${sysconfdir}/inittab ${sysconfdir}/init.d/start_getty"
-CONFFILES_${PN} = "${sysconfdir}/inittab"
 
 USE_VT ?= "1"
 SYSVINIT_ENABLED_GETTYS ?= "1"
diff --git a/meta/recipes-core/udev/udev-extraconf_1.1.bb b/meta/recipes-core/udev/udev-extraconf_1.1.bb
index ecd4a8a..36aeafa 100644
--- a/meta/recipes-core/udev/udev-extraconf_1.1.bb
+++ b/meta/recipes-core/udev/udev-extraconf_1.1.bb
@@ -34,7 +34,6 @@ do_install() {
 
 FILES_${PN} = "${sysconfdir}/udev"
 RDEPENDS_${PN} = "udev"
-CONFFILES_${PN} = "${sysconfdir}/udev/mount.blacklist"
 
 # to replace udev-extra-rules from meta-oe
 RPROVIDES_${PN} = "udev-extra-rules"
-- 
1.7.9.5

[PATCH 6/8] recipes-devtools: cleanup CONFFILES

[Robert Yang]

The ${sysconfdir}/* are CONFFILES by default.

[YOCTO #8436]

Signed-off-by: Robert Yang <liezhi.yang@windriver.com>
---
 meta/recipes-devtools/opkg/opkg-arch-config_1.0.bb |    2 --
 meta/recipes-devtools/opkg/opkg_0.3.1.bb           |    1 -
 2 files changed, 3 deletions(-)

diff --git a/meta/recipes-devtools/opkg/opkg-arch-config_1.0.bb b/meta/recipes-devtools/opkg/opkg-arch-config_1.0.bb
index ace3de4..861f011 100644
--- a/meta/recipes-devtools/opkg/opkg-arch-config_1.0.bb
+++ b/meta/recipes-devtools/opkg/opkg-arch-config_1.0.bb
@@ -28,8 +28,6 @@ do_install () {
 
 FILES_${PN} = "${sysconfdir}/opkg/ "
 
-CONFFILES_${PN} += "${sysconfdir}/opkg/arch.conf"
-
 RREPLACES_${PN} = "opkg-config-base"
 RCONFLICTS_${PN} = "opkg-config-base"
 RPROVIDES_${PN} = "opkg-config-base"
diff --git a/meta/recipes-devtools/opkg/opkg_0.3.1.bb b/meta/recipes-devtools/opkg/opkg_0.3.1.bb
index 577f495..345b555 100644
--- a/meta/recipes-devtools/opkg/opkg_0.3.1.bb
+++ b/meta/recipes-devtools/opkg/opkg_0.3.1.bb
@@ -72,4 +72,3 @@ FILES_${PN} += "${systemd_unitdir}/system/"
 
 BBCLASSEXTEND = "native nativesdk"
 
-CONFFILES_${PN} = "${sysconfdir}/opkg/opkg.conf"
-- 
1.7.9.5

[PATCH 7/8] recipes-graphics: cleanup CONFFILES

[Robert Yang]

The ${sysconfdir}/* are CONFFILES by default.

[YOCTO #8436]

Signed-off-by: Robert Yang <liezhi.yang@windriver.com>
---
 .../xinput-calibrator/pointercal-xinput_0.0.bb     |    1 -
 .../xorg-xserver/xserver-xf86-config_0.1.bb        |    3 ---
 2 files changed, 4 deletions(-)

diff --git a/meta/recipes-graphics/xinput-calibrator/pointercal-xinput_0.0.bb b/meta/recipes-graphics/xinput-calibrator/pointercal-xinput_0.0.bb
index 65348c3..9b52505 100644
--- a/meta/recipes-graphics/xinput-calibrator/pointercal-xinput_0.0.bb
+++ b/meta/recipes-graphics/xinput-calibrator/pointercal-xinput_0.0.bb
@@ -17,4 +17,3 @@ do_install() {
 }
 
 PACKAGE_ARCH = "${MACHINE_ARCH}"
-CONFFILES_${PN} = "${sysconfdir}/pointercal.xinput"
diff --git a/meta/recipes-graphics/xorg-xserver/xserver-xf86-config_0.1.bb b/meta/recipes-graphics/xorg-xserver/xserver-xf86-config_0.1.bb
index e07c204..aa5099c 100644
--- a/meta/recipes-graphics/xorg-xserver/xserver-xf86-config_0.1.bb
+++ b/meta/recipes-graphics/xorg-xserver/xserver-xf86-config_0.1.bb
@@ -13,9 +13,6 @@ SRC_URI_append_libc-musl = "\
 
 S = "${WORKDIR}"
 
-CONFFILES_${PN} = "${sysconfdir}/X11/xorg.conf"
-CONFFILES_${PN}_append_libc-musl = " ${sysconfdir}/X11/xorg.conf.d/10-preload-modules.conf"
-
 PACKAGE_ARCH = "${MACHINE_ARCH}"
 ALLOW_EMPTY_${PN} = "1"
 
-- 
1.7.9.5

[PATCH 8/8] recipes-multimedia/sato/support: cleanup CONFFILES

[Robert Yang]

The ${sysconfdir}/* are CONFFILES by default.

[YOCTO #8436]

Signed-off-by: Robert Yang <liezhi.yang@windriver.com>
---
 meta/recipes-multimedia/pulseaudio/pulseaudio.inc  |    8 --------
 .../pulseaudio-client-conf-sato_1.bb               |    1 -
 .../ca-certificates/ca-certificates_20150426.bb    |    4 ----
 3 files changed, 13 deletions(-)

diff --git a/meta/recipes-multimedia/pulseaudio/pulseaudio.inc b/meta/recipes-multimedia/pulseaudio/pulseaudio.inc
index ec73526..21853b5 100644
--- a/meta/recipes-multimedia/pulseaudio/pulseaudio.inc
+++ b/meta/recipes-multimedia/pulseaudio/pulseaudio.inc
@@ -124,14 +124,6 @@ FILES_${PN}-misc = "${bindir}/* ${libdir}/pulseaudio/libpulsedsp.so"
 # Allow the pulseaudio package to be created empty as a placeholder (-dbg and -dev depend on it)
 ALLOW_EMPTY_${PN} = "1"
 
-CONFFILES_libpulse = "${sysconfdir}/pulse/client.conf"
-
-CONFFILES_pulseaudio-server = "\ 
-  ${sysconfdir}/pulse/default.pa \
-  ${sysconfdir}/pulse/daemon.conf \
-  ${sysconfdir}/pulse/system.pa \
-  "
-
 pkg_postinst_${PN}-server() {
         if [ -z "$D" ] && [ -e ${sysconfdir}/init.d/populate-volatile.sh ] ; then
             ${sysconfdir}/init.d/populate-volatile.sh update
diff --git a/meta/recipes-sato/pulseaudio-sato/pulseaudio-client-conf-sato_1.bb b/meta/recipes-sato/pulseaudio-sato/pulseaudio-client-conf-sato_1.bb
index e27339f..2ccda38 100644
--- a/meta/recipes-sato/pulseaudio-sato/pulseaudio-client-conf-sato_1.bb
+++ b/meta/recipes-sato/pulseaudio-sato/pulseaudio-client-conf-sato_1.bb
@@ -14,4 +14,3 @@ do_install() {
 }
 
 FILES_${PN} = "${sysconfdir}/pulse/client.conf.d/50-sato.conf"
-CONFFILES_${PN} = "${sysconfdir}/pulse/client.conf.d/50-sato.conf"
diff --git a/meta/recipes-support/ca-certificates/ca-certificates_20150426.bb b/meta/recipes-support/ca-certificates/ca-certificates_20150426.bb
index 716e3df..aca34a4 100644
--- a/meta/recipes-support/ca-certificates/ca-certificates_20150426.bb
+++ b/meta/recipes-support/ca-certificates/ca-certificates_20150426.bb
@@ -65,10 +65,6 @@ pkg_postinst_${PN} () {
     SYSROOT="$D" update-ca-certificates
 }
 
-CONFFILES_${PN} += "${sysconfdir}/ca-certificates.conf"
-
-# Postinsts don't seem to be run for nativesdk packages when populating SDKs.
-CONFFILES_${PN}_append_class-nativesdk = " ${sysconfdir}/ssl/certs/ca-certificates.crt"
 do_install_append_class-nativesdk () {
     SYSROOT="${D}${SDKPATHNATIVE}" update-ca-certificates
 }
-- 
1.7.9.5

Re: [PATCH 1/8] bitbake.conf: set CONFFILES by default

[Mike Looijmans]

How about /etc/init.d and /etc/rcS.d? Are these now suddenly CONFFILES too?

On 15-01-16 03:29, Robert Yang wrote:
> There are a lot of binary packages have files in /etc/, but only a few
> of them have set CONFFILES, more than 180 packages are not set in a
> world build.  So treat all the files in /etc/ as CONFFILES as Debian
> does:
> - All the files in /etc/ are CONFFILES
> - Move the file out of /etc/ if it is not a conffile
> - If /etc/foo can't be moved out, and is changed during
>    runtime, then create a symlink /etc/foo -> /var/foo
>
> [YOCTO #8436]
>
> Signed-off-by: Robert Yang <liezhi.yang@windriver.com>
> ---
>   meta/conf/bitbake.conf |    1 +
>   1 file changed, 1 insertion(+)
>
> diff --git a/meta/conf/bitbake.conf b/meta/conf/bitbake.conf
> index 2fe6b24..f0e4b3a 100644
> --- a/meta/conf/bitbake.conf
> +++ b/meta/conf/bitbake.conf
> @@ -271,6 +271,7 @@ PACKAGE_BEFORE_PN ?= ""
>   PACKAGES = "${PN}-dbg ${PN}-staticdev ${PN}-dev ${PN}-doc ${PN}-locale ${PACKAGE_BEFORE_PN} ${PN}"
>   PACKAGES_DYNAMIC = "^${PN}-locale-.*"
>   FILES = ""
> +CONFFILES = "${sysconfdir}/*"
>
>   FILES_${PN} = "${bindir}/* ${sbindir}/* ${libexecdir}/* ${libdir}/lib*${SOLIBS} \
>               ${sysconfdir} ${sharedstatedir} ${localstatedir} \
>



Kind regards,

Mike Looijmans
System Expert

TOPIC Embedded Products
Eindhovenseweg 32-C, NL-5683 KH Best
Postbus 440, NL-5680 AK Best
Telefoon: +31 (0) 499 33 69 79
E-mail: mike.looijmans@topicproducts.com
Website: www.topicproducts.com

Please consider the environment before printing this e-mail

Re: [PATCH 1/8] bitbake.conf: set CONFFILES by default

[Robert Yang]

On 01/15/2016 02:27 PM, Mike Looijmans wrote:
> How about /etc/init.d and /etc/rcS.d? Are these now suddenly CONFFILES too?

For /etc/init.d/*, yes, and I think that they should be CONFFILES.

For /etc/rcS.d/*, they are symlinks which are not owned by pkg,
but created by update-rc.d.

// Robert

>
> On 15-01-16 03:29, Robert Yang wrote:
>> There are a lot of binary packages have files in /etc/, but only a few
>> of them have set CONFFILES, more than 180 packages are not set in a
>> world build.  So treat all the files in /etc/ as CONFFILES as Debian
>> does:
>> - All the files in /etc/ are CONFFILES
>> - Move the file out of /etc/ if it is not a conffile
>> - If /etc/foo can't be moved out, and is changed during
>>    runtime, then create a symlink /etc/foo -> /var/foo
>>
>> [YOCTO #8436]
>>
>> Signed-off-by: Robert Yang <liezhi.yang@windriver.com>
>> ---
>>   meta/conf/bitbake.conf |    1 +
>>   1 file changed, 1 insertion(+)
>>
>> diff --git a/meta/conf/bitbake.conf b/meta/conf/bitbake.conf
>> index 2fe6b24..f0e4b3a 100644
>> --- a/meta/conf/bitbake.conf
>> +++ b/meta/conf/bitbake.conf
>> @@ -271,6 +271,7 @@ PACKAGE_BEFORE_PN ?= ""
>>   PACKAGES = "${PN}-dbg ${PN}-staticdev ${PN}-dev ${PN}-doc ${PN}-locale
>> ${PACKAGE_BEFORE_PN} ${PN}"
>>   PACKAGES_DYNAMIC = "^${PN}-locale-.*"
>>   FILES = ""
>> +CONFFILES = "${sysconfdir}/*"
>>
>>   FILES_${PN} = "${bindir}/* ${sbindir}/* ${libexecdir}/*
>> ${libdir}/lib*${SOLIBS} \
>>               ${sysconfdir} ${sharedstatedir} ${localstatedir} \
>>
>
>
>
> Kind regards,
>
> Mike Looijmans
> System Expert
>
> TOPIC Embedded Products
> Eindhovenseweg 32-C, NL-5683 KH Best
> Postbus 440, NL-5680 AK Best
> Telefoon: +31 (0) 499 33 69 79
> E-mail: mike.looijmans@topicproducts.com
> Website: www.topicproducts.com
>
> Please consider the environment before printing this e-mail
>
>
>
>
>
>

Re: [PATCH 1/8] bitbake.conf: set CONFFILES by default

[Richard Purdie]

On Thu, 2016-01-14 at 18:29 -0800, Robert Yang wrote:
> There are a lot of binary packages have files in /etc/, but only a
> few
> of them have set CONFFILES, more than 180 packages are not set in a
> world build.  So treat all the files in /etc/ as CONFFILES as Debian
> does:
> - All the files in /etc/ are CONFFILES
> - Move the file out of /etc/ if it is not a conffile
> - If /etc/foo can't be moved out, and is changed during
>   runtime, then create a symlink /etc/foo -> /var/foo
> 
> [YOCTO #8436]
> 
> Signed-off-by: Robert Yang <liezhi.yang@windriver.com>
> ---
>  meta/conf/bitbake.conf |    1 +
>  1 file changed, 1 insertion(+)

We have discussed this before and we decided that setting CONFFILES
globally seemed like a bad idea?

What has changed since the last discussion?

Cheers,

Richard

Re: [PATCH 1/8] bitbake.conf: set CONFFILES by default

[Robert Yang]

On 01/15/2016 07:12 PM, Richard Purdie wrote:
> On Thu, 2016-01-14 at 18:29 -0800, Robert Yang wrote:
>> There are a lot of binary packages have files in /etc/, but only a
>> few
>> of them have set CONFFILES, more than 180 packages are not set in a
>> world build.  So treat all the files in /etc/ as CONFFILES as Debian
>> does:
>> - All the files in /etc/ are CONFFILES
>> - Move the file out of /etc/ if it is not a conffile
>> - If /etc/foo can't be moved out, and is changed during
>>    runtime, then create a symlink /etc/foo -> /var/foo
>>
>> [YOCTO #8436]
>>
>> Signed-off-by: Robert Yang <liezhi.yang@windriver.com>
>> ---
>>   meta/conf/bitbake.conf |    1 +
>>   1 file changed, 1 insertion(+)
>
> We have discussed this before and we decided that setting CONFFILES
> globally seemed like a bad idea?

Sorry, but I can't find any emails which said this was a bad idea.
If we don't set CONFFILES by default, maybe we need mark a lot of
CONFFILES inside the recipes. I think that mark the CONFFILES under
/etc/ doesn't hurt anything, but help a lot when doing upgrade on
target:

1) When the old file is not changed, it will be replaced by new file.
2) When the old file is changed, it will ask the user what to do (dpkg)
    or save a bak (rpm).

//Robert

>
> What has changed since the last discussion?
>
> Cheers,
>
> Richard
>

Re: [PATCH 1/8] bitbake.conf: set CONFFILES by default

[Mike Looijmans]

On 15-01-16 07:37, Robert Yang wrote:
>
>
> On 01/15/2016 02:27 PM, Mike Looijmans wrote:
>> How about /etc/init.d and /etc/rcS.d? Are these now suddenly CONFFILES
>> too?
>
> For /etc/init.d/*, yes, and I think that they should be CONFFILES.

Startup scripts are config files? Wow that's news to me.

They all include /etc/defaults/something for a good reason: So you don't 
have to hack the script.

> For /etc/rcS.d/*, they are symlinks which are not owned by pkg,
> but created by update-rc.d.

Only if the recipe happens to use update-rc. Quite some packages don't 
use it and just create the symlinks themselves.


As I recall, this suggestion has been made before and was rejected then. 
Nothing really changed since.


-- 
Mike Looijmans

Re: [PATCH 1/8] bitbake.conf: set CONFFILES by default

[Mike Looijmans]

On 16-01-16 06:36, Robert Yang wrote:
>
>
> On 01/15/2016 07:12 PM, Richard Purdie wrote:
>> On Thu, 2016-01-14 at 18:29 -0800, Robert Yang wrote:
>>> There are a lot of binary packages have files in /etc/, but only a
>>> few
>>> of them have set CONFFILES, more than 180 packages are not set in a
>>> world build.  So treat all the files in /etc/ as CONFFILES as Debian
>>> does:
>>> - All the files in /etc/ are CONFFILES
>>> - Move the file out of /etc/ if it is not a conffile
>>> - If /etc/foo can't be moved out, and is changed during
>>>    runtime, then create a symlink /etc/foo -> /var/foo
>>>
>>> [YOCTO #8436]
>>>
>>> Signed-off-by: Robert Yang <liezhi.yang@windriver.com>
>>> ---
>>>   meta/conf/bitbake.conf |    1 +
>>>   1 file changed, 1 insertion(+)
>>
>> We have discussed this before and we decided that setting CONFFILES
>> globally seemed like a bad idea?

That is how I recall it. Please please please don't do it. It will wreak 
havoc on all kinds of systems.

> Sorry, but I can't find any emails which said this was a bad idea.
> If we don't set CONFFILES by default, maybe we need mark a lot of
> CONFFILES inside the recipes. I think that mark the CONFFILES under
> /etc/ doesn't hurt anything, but help a lot when doing upgrade on
> target:
>
> 1) When the old file is not changed, it will be replaced by new file.
> 2) When the old file is changed, it will ask the user what to do (dpkg)
>     or save a bak (rpm).

In a perfect world, yeah.

In the world other people have to live in, the GUI has to shut down to 
free up resources to get the upgrade to complete successfully. This 
means you cannot ask the user sitting on the couch holding the remote 
staring at a static "please wait upgrade in progress" screen any questions.

Then if you could ask him questions, he'll just have no clue as to what 
to answer.

You can create "backup" files all you like, but 99% of the users will 
never ever see the file system, and do not know how to rename, edit or 
copy a file. Which on the other hand does not stop them from trying, and 
they're quite used to the situation that a simple reinstall will bring 
things back in working order if they somehow "got it wrong".

In the world I live in, things happen that change files that shouldn't 
be changed. Power outage (could be just the cleaning lady pulling the 
plug) just to name one.

Manually fixing things up is fine if you're an experienced developer or 
power user. But remember that the majority of the end users are less 
tech savvy, and just use a "box" with software that happens to be built 
by OpenEmbedded, which they never heard of.

>> What has changed since the last discussion?

Nothing relevant...


-- 
Mike Looijmans

Re: [PATCH 1/8] bitbake.conf: set CONFFILES by default

[Robert Yang]

On 01/16/2016 06:51 PM, Mike Looijmans wrote:
> On 15-01-16 07:37, Robert Yang wrote:
>>
>>
>> On 01/15/2016 02:27 PM, Mike Looijmans wrote:
>>> How about /etc/init.d and /etc/rcS.d? Are these now suddenly CONFFILES
>>> too?
>>
>> For /etc/init.d/*, yes, and I think that they should be CONFFILES.
>
> Startup scripts are config files? Wow that's news to me.

Yes, debian treats startup scripts as config files.

>
> They all include /etc/defaults/something for a good reason: So you don't have to
> hack the script.
>
>> For /etc/rcS.d/*, they are symlinks which are not owned by pkg,
>> but created by update-rc.d.
>
> Only if the recipe happens to use update-rc. Quite some packages don't use it
> and just create the symlinks themselves.
>
>
> As I recall, this suggestion has been made before and was rejected then. Nothing
> really changed since.

I had sent this patch before, but I didn't see any rejected emails.

// Robert

>
>

Re: [PATCH 1/8] bitbake.conf: set CONFFILES by default

[Robert Yang]

On 01/16/2016 07:07 PM, Mike Looijmans wrote:
> On 16-01-16 06:36, Robert Yang wrote:
>>
>>
>> On 01/15/2016 07:12 PM, Richard Purdie wrote:
>>> On Thu, 2016-01-14 at 18:29 -0800, Robert Yang wrote:
>>>> There are a lot of binary packages have files in /etc/, but only a
>>>> few
>>>> of them have set CONFFILES, more than 180 packages are not set in a
>>>> world build.  So treat all the files in /etc/ as CONFFILES as Debian
>>>> does:
>>>> - All the files in /etc/ are CONFFILES
>>>> - Move the file out of /etc/ if it is not a conffile
>>>> - If /etc/foo can't be moved out, and is changed during
>>>>    runtime, then create a symlink /etc/foo -> /var/foo
>>>>
>>>> [YOCTO #8436]
>>>>
>>>> Signed-off-by: Robert Yang <liezhi.yang@windriver.com>
>>>> ---
>>>>   meta/conf/bitbake.conf |    1 +
>>>>   1 file changed, 1 insertion(+)
>>>
>>> We have discussed this before and we decided that setting CONFFILES
>>> globally seemed like a bad idea?
>
> That is how I recall it. Please please please don't do it. It will wreak havoc
> on all kinds of systems.
>
>> Sorry, but I can't find any emails which said this was a bad idea.
>> If we don't set CONFFILES by default, maybe we need mark a lot of
>> CONFFILES inside the recipes. I think that mark the CONFFILES under
>> /etc/ doesn't hurt anything, but help a lot when doing upgrade on
>> target:
>>
>> 1) When the old file is not changed, it will be replaced by new file.
>> 2) When the old file is changed, it will ask the user what to do (dpkg)
>>     or save a bak (rpm).
>
> In a perfect world, yeah.
>
> In the world other people have to live in, the GUI has to shut down to free up
> resources to get the upgrade to complete successfully. This means you cannot ask
> the user sitting on the couch holding the remote staring at a static "please
> wait upgrade in progress" screen any questions.
>
> Then if you could ask him questions, he'll just have no clue as to what to answer.
>
> You can create "backup" files all you like, but 99% of the users will never ever
> see the file system, and do not know how to rename, edit or copy a file. Which
> on the other hand does not stop them from trying, and they're quite used to the
> situation that a simple reinstall will bring things back in working order if
> they somehow "got it wrong".
>
> In the world I live in, things happen that change files that shouldn't be
> changed. Power outage (could be just the cleaning lady pulling the plug) just to
> name one.
>
> Manually fixing things up is fine if you're an experienced developer or power
> user. But remember that the majority of the end users are less tech savvy, and
> just use a "box" with software that happens to be built by OpenEmbedded, which
> they never heard of.


For apt-get, you can configure dpkg to use non-interactive when upgrade or
dist-upgrade:
--force-confnew or --force-confold

For ipkg, I think that you can use: --force-maintainer

For rpm, save a bak is the default action.

// Robert

>
>>> What has changed since the last discussion?
>
> Nothing relevant...
>
>