* [PATCH] opkg: create opkg.lock in /run instead of /var/run
@ 2015-09-21 11:09 Jonathan Liu
2015-09-21 12:02 ` Jussi Kukkonen
2015-09-21 12:16 ` Anders Darander
0 siblings, 2 replies; 6+ messages in thread
From: Jonathan Liu @ 2015-09-21 11:09 UTC (permalink / raw)
To: openembedded-core
This avoids a "Could not unlink" warning when extracting a /var/run
symbolic link pointing to /run from the base-files package as it is
unable to unlink the /var/run directory when it contains opkg.lock.
This also fixes an issue where /var/run is created as a directory
instead of a symbolic link to /run.
Signed-off-by: Jonathan Liu <net147@gmail.com>
---
...reate-opkg.lock-in-run-instead-of-var-run.patch | 34 ++++++++++++++++++++++
meta/recipes-devtools/opkg/opkg_0.3.0.bb | 1 +
2 files changed, 35 insertions(+)
create mode 100644 meta/recipes-devtools/opkg/opkg/0001-opkg_conf-create-opkg.lock-in-run-instead-of-var-run.patch
diff --git a/meta/recipes-devtools/opkg/opkg/0001-opkg_conf-create-opkg.lock-in-run-instead-of-var-run.patch b/meta/recipes-devtools/opkg/opkg/0001-opkg_conf-create-opkg.lock-in-run-instead-of-var-run.patch
new file mode 100644
index 0000000..255021b
--- /dev/null
+++ b/meta/recipes-devtools/opkg/opkg/0001-opkg_conf-create-opkg.lock-in-run-instead-of-var-run.patch
@@ -0,0 +1,34 @@
+From a4628a6171f393add9a2b287483ca39bb72b4dd6 Mon Sep 17 00:00:00 2001
+From: Jonathan Liu <net147@gmail.com>
+Date: Mon, 21 Sep 2015 20:23:23 +1000
+Subject: [PATCH] opkg_conf: create opkg.lock in /run instead of /var/run
+
+This avoids a "Could not unlink" warning when extracting a /var/run
+symbolic link pointing to /run from a package as it is unable to
+unlink the /var/run directory when it contains opkg.lock.
+
+This also fixes an issue where /var/run is created as a directory
+instead of a symbolic link to /run.
+
+Upstream-Status: Inappropriate [OE-Specific]
+Signed-off-by: Jonathan Liu <net147@gmail.com>
+---
+ libopkg/opkg_conf.h | 2 +-
+ 1 file changed, 1 insertion(+), 1 deletion(-)
+
+diff --git a/libopkg/opkg_conf.h b/libopkg/opkg_conf.h
+index 7bca948..5a1bc44 100644
+--- a/libopkg/opkg_conf.h
++++ b/libopkg/opkg_conf.h
+@@ -40,7 +40,7 @@ extern "C" {
+ #define OPKG_CONF_DEFAULT_STATUS_FILE "/var/lib/opkg/status"
+ #define OPKG_CONF_DEFAULT_CACHE_DIR "/var/cache/opkg"
+ #define OPKG_CONF_DEFAULT_CONF_FILE_DIR "/etc/opkg"
+-#define OPKG_CONF_DEFAULT_LOCK_FILE "/var/run/opkg.lock"
++#define OPKG_CONF_DEFAULT_LOCK_FILE "/run/opkg.lock"
+
+ /* In case the config file defines no dest */
+ #define OPKG_CONF_DEFAULT_DEST_NAME "root"
+--
+2.5.0
+
diff --git a/meta/recipes-devtools/opkg/opkg_0.3.0.bb b/meta/recipes-devtools/opkg/opkg_0.3.0.bb
index f4dbb2d..08fb62c 100644
--- a/meta/recipes-devtools/opkg/opkg_0.3.0.bb
+++ b/meta/recipes-devtools/opkg/opkg_0.3.0.bb
@@ -15,6 +15,7 @@ SRC_URI = "http://downloads.yoctoproject.org/releases/${BPN}/${BPN}-${PV}.tar.gz
file://opkg-configure.service \
file://opkg.conf \
file://0001-opkg_archive-add-support-for-empty-compressed-files.patch \
+ file://0001-opkg_conf-create-opkg.lock-in-run-instead-of-var-run.patch \
"
SRC_URI[md5sum] = "3412cdc71d78b98facc84b19331ec64e"
--
2.5.0
^ permalink raw reply related [flat|nested] 6+ messages in thread* Re: [PATCH] opkg: create opkg.lock in /run instead of /var/run
2015-09-21 11:09 [PATCH] opkg: create opkg.lock in /run instead of /var/run Jonathan Liu
@ 2015-09-21 12:02 ` Jussi Kukkonen
2015-09-21 12:06 ` Jonathan Liu
2015-09-21 12:16 ` Anders Darander
1 sibling, 1 reply; 6+ messages in thread
From: Jussi Kukkonen @ 2015-09-21 12:02 UTC (permalink / raw)
To: Jonathan Liu; +Cc: Patches and discussions about the oe-core layer
[-- Attachment #1: Type: text/plain, Size: 3572 bytes --]
On 21 September 2015 at 14:09, Jonathan Liu <net147@gmail.com> wrote:
> This avoids a "Could not unlink" warning when extracting a /var/run
> symbolic link pointing to /run from the base-files package as it is
> unable to unlink the /var/run directory when it contains opkg.lock.
>
> This also fixes an issue where /var/run is created as a directory
> instead of a symbolic link to /run.
>
> Signed-off-by: Jonathan Liu <net147@gmail.com>
> ---
> ...reate-opkg.lock-in-run-instead-of-var-run.patch | 34
> ++++++++++++++++++++++
> meta/recipes-devtools/opkg/opkg_0.3.0.bb | 1 +
> 2 files changed, 35 insertions(+)
> create mode 100644
> meta/recipes-devtools/opkg/opkg/0001-opkg_conf-create-opkg.lock-in-run-instead-of-var-run.patch
>
> diff --git
> a/meta/recipes-devtools/opkg/opkg/0001-opkg_conf-create-opkg.lock-in-run-instead-of-var-run.patch
> b/meta/recipes-devtools/opkg/opkg/0001-opkg_conf-create-opkg.lock-in-run-instead-of-var-run.patch
> new file mode 100644
> index 0000000..255021b
> --- /dev/null
> +++
> b/meta/recipes-devtools/opkg/opkg/0001-opkg_conf-create-opkg.lock-in-run-instead-of-var-run.patch
> @@ -0,0 +1,34 @@
> +From a4628a6171f393add9a2b287483ca39bb72b4dd6 Mon Sep 17 00:00:00 2001
> +From: Jonathan Liu <net147@gmail.com>
> +Date: Mon, 21 Sep 2015 20:23:23 +1000
> +Subject: [PATCH] opkg_conf: create opkg.lock in /run instead of /var/run
> +
> +This avoids a "Could not unlink" warning when extracting a /var/run
> +symbolic link pointing to /run from a package as it is unable to
> +unlink the /var/run directory when it contains opkg.lock.
> +
> +This also fixes an issue where /var/run is created as a directory
> +instead of a symbolic link to /run.
> +
> +Upstream-Status: Inappropriate [OE-Specific]
>
Isn't using /run for this appropriate for everyone? I would have expected
this to be an uncontroversial change.
Jussi
+Signed-off-by: Jonathan Liu <net147@gmail.com>
> +---
> + libopkg/opkg_conf.h | 2 +-
> + 1 file changed, 1 insertion(+), 1 deletion(-)
> +
> +diff --git a/libopkg/opkg_conf.h b/libopkg/opkg_conf.h
> +index 7bca948..5a1bc44 100644
> +--- a/libopkg/opkg_conf.h
> ++++ b/libopkg/opkg_conf.h
> +@@ -40,7 +40,7 @@ extern "C" {
> + #define OPKG_CONF_DEFAULT_STATUS_FILE "/var/lib/opkg/status"
> + #define OPKG_CONF_DEFAULT_CACHE_DIR "/var/cache/opkg"
> + #define OPKG_CONF_DEFAULT_CONF_FILE_DIR "/etc/opkg"
> +-#define OPKG_CONF_DEFAULT_LOCK_FILE "/var/run/opkg.lock"
> ++#define OPKG_CONF_DEFAULT_LOCK_FILE "/run/opkg.lock"
> +
> + /* In case the config file defines no dest */
> + #define OPKG_CONF_DEFAULT_DEST_NAME "root"
> +--
> +2.5.0
> +
> diff --git a/meta/recipes-devtools/opkg/opkg_0.3.0.bb
> b/meta/recipes-devtools/opkg/opkg_0.3.0.bb
> index f4dbb2d..08fb62c 100644
> --- a/meta/recipes-devtools/opkg/opkg_0.3.0.bb
> +++ b/meta/recipes-devtools/opkg/opkg_0.3.0.bb
> @@ -15,6 +15,7 @@ SRC_URI = "
> http://downloads.yoctoproject.org/releases/${BPN}/${BPN}-${PV}.tar.gz
> file://opkg-configure.service \
> file://opkg.conf \
>
> file://0001-opkg_archive-add-support-for-empty-compressed-files.patch \
> +
> file://0001-opkg_conf-create-opkg.lock-in-run-instead-of-var-run.patch \
> "
>
> SRC_URI[md5sum] = "3412cdc71d78b98facc84b19331ec64e"
> --
> 2.5.0
>
> --
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core
>
[-- Attachment #2: Type: text/html, Size: 5167 bytes --]
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [PATCH] opkg: create opkg.lock in /run instead of /var/run
2015-09-21 12:02 ` Jussi Kukkonen
@ 2015-09-21 12:06 ` Jonathan Liu
0 siblings, 0 replies; 6+ messages in thread
From: Jonathan Liu @ 2015-09-21 12:06 UTC (permalink / raw)
To: Jussi Kukkonen; +Cc: Patches and discussions about the oe-core layer
[-- Attachment #1: Type: text/plain, Size: 4512 bytes --]
On 21/09/2015 10:02 PM, Jussi Kukkonen wrote:
> On 21 September 2015 at 14:09, Jonathan Liu <net147@gmail.com
> <mailto:net147@gmail.com>> wrote:
>
> This avoids a "Could not unlink" warning when extracting a /var/run
> symbolic link pointing to /run from the base-files package as it is
> unable to unlink the /var/run directory when it contains opkg.lock.
>
> This also fixes an issue where /var/run is created as a directory
> instead of a symbolic link to /run.
>
> Signed-off-by: Jonathan Liu <net147@gmail.com
> <mailto:net147@gmail.com>>
> ---
> ...reate-opkg.lock-in-run-instead-of-var-run.patch | 34
> ++++++++++++++++++++++
> meta/recipes-devtools/opkg/opkg_0.3.0.bb <http://opkg_0.3.0.bb>
> | 1 +
> 2 files changed, 35 insertions(+)
> create mode 100644
> meta/recipes-devtools/opkg/opkg/0001-opkg_conf-create-opkg.lock-in-run-instead-of-var-run.patch
>
> diff --git
> a/meta/recipes-devtools/opkg/opkg/0001-opkg_conf-create-opkg.lock-in-run-instead-of-var-run.patch
> b/meta/recipes-devtools/opkg/opkg/0001-opkg_conf-create-opkg.lock-in-run-instead-of-var-run.patch
> new file mode 100644
> index 0000000..255021b
> --- /dev/null
> +++
> b/meta/recipes-devtools/opkg/opkg/0001-opkg_conf-create-opkg.lock-in-run-instead-of-var-run.patch
> @@ -0,0 +1,34 @@
> +From a4628a6171f393add9a2b287483ca39bb72b4dd6 Mon Sep 17 00:00:00
> 2001
> +From: Jonathan Liu <net147@gmail.com <mailto:net147@gmail.com>>
> +Date: Mon, 21 Sep 2015 20:23:23 +1000
> +Subject: [PATCH] opkg_conf: create opkg.lock in /run instead of
> /var/run
> +
> +This avoids a "Could not unlink" warning when extracting a /var/run
> +symbolic link pointing to /run from a package as it is unable to
> +unlink the /var/run directory when it contains opkg.lock.
> +
> +This also fixes an issue where /var/run is created as a directory
> +instead of a symbolic link to /run.
> +
> +Upstream-Status: Inappropriate [OE-Specific]
>
>
> Isn't using /run for this appropriate for everyone? I would have
> expected this to be an uncontroversial change.
>
> Jussi
I didn't want to create a behavior change for other projects using opkg
that aren't necessarily following the FHS 3 Draft.
Regards,
Jonathan
>
>
> +Signed-off-by: Jonathan Liu <net147@gmail.com
> <mailto:net147@gmail.com>>
> +---
> + libopkg/opkg_conf.h | 2 +-
> + 1 file changed, 1 insertion(+), 1 deletion(-)
> +
> +diff --git a/libopkg/opkg_conf.h b/libopkg/opkg_conf.h
> +index 7bca948..5a1bc44 100644
> +--- a/libopkg/opkg_conf.h
> ++++ b/libopkg/opkg_conf.h
> +@@ -40,7 +40,7 @@ extern "C" {
> + #define OPKG_CONF_DEFAULT_STATUS_FILE "/var/lib/opkg/status"
> + #define OPKG_CONF_DEFAULT_CACHE_DIR "/var/cache/opkg"
> + #define OPKG_CONF_DEFAULT_CONF_FILE_DIR "/etc/opkg"
> +-#define OPKG_CONF_DEFAULT_LOCK_FILE "/var/run/opkg.lock"
> ++#define OPKG_CONF_DEFAULT_LOCK_FILE "/run/opkg.lock"
> +
> + /* In case the config file defines no dest */
> + #define OPKG_CONF_DEFAULT_DEST_NAME "root"
> +--
> +2.5.0
> +
> diff --git a/meta/recipes-devtools/opkg/opkg_0.3.0.bb
> <http://opkg_0.3.0.bb> b/meta/recipes-devtools/opkg/opkg_0.3.0.bb
> <http://opkg_0.3.0.bb>
> index f4dbb2d..08fb62c 100644
> --- a/meta/recipes-devtools/opkg/opkg_0.3.0.bb <http://opkg_0.3.0.bb>
> +++ b/meta/recipes-devtools/opkg/opkg_0.3.0.bb <http://opkg_0.3.0.bb>
> @@ -15,6 +15,7 @@ SRC_URI =
> "http://downloads.yoctoproject.org/releases/${BPN}/${BPN}-${PV}.tar.gz
> <http://downloads.yoctoproject.org/releases/$%7BBPN%7D/$%7BBPN%7D-$%7BPV%7D.tar.gz>
> file://opkg-configure.service \
> file://opkg.conf \
> file://0001-opkg_archive-add-support-for-empty-compressed-files.patch
> \
> +
> file://0001-opkg_conf-create-opkg.lock-in-run-instead-of-var-run.patch
> \
> "
>
> SRC_URI[md5sum] = "3412cdc71d78b98facc84b19331ec64e"
> --
> 2.5.0
>
> --
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> <mailto:Openembedded-core@lists.openembedded.org>
> http://lists.openembedded.org/mailman/listinfo/openembedded-core
>
>
[-- Attachment #2: Type: text/html, Size: 8589 bytes --]
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] opkg: create opkg.lock in /run instead of /var/run
2015-09-21 11:09 [PATCH] opkg: create opkg.lock in /run instead of /var/run Jonathan Liu
2015-09-21 12:02 ` Jussi Kukkonen
@ 2015-09-21 12:16 ` Anders Darander
2015-09-25 15:06 ` Jonathan Liu
1 sibling, 1 reply; 6+ messages in thread
From: Anders Darander @ 2015-09-21 12:16 UTC (permalink / raw)
To: openembedded-core
* Jonathan Liu <net147@gmail.com> [150921 13:08]:
> This avoids a "Could not unlink" warning when extracting a /var/run
> symbolic link pointing to /run from the base-files package as it is
> unable to unlink the /var/run directory when it contains opkg.lock.
> This also fixes an issue where /var/run is created as a directory
> instead of a symbolic link to /run.
This fixes my recently discoverd runtime issues, when forwardporting a
systemd-based distro from daisy/dizzy to the current master.
> Signed-off-by: Jonathan Liu <net147@gmail.com>
Acked-by: Anders Darander <anders@chargestorm.se>
--
Anders Darander
ChargeStorm AB / eStorm AB
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] opkg: create opkg.lock in /run instead of /var/run
2015-09-21 12:16 ` Anders Darander
@ 2015-09-25 15:06 ` Jonathan Liu
2015-09-28 6:42 ` Anders Darander
0 siblings, 1 reply; 6+ messages in thread
From: Jonathan Liu @ 2015-09-25 15:06 UTC (permalink / raw)
To: openembedded-core
On 21/09/2015 10:16 PM, Anders Darander wrote:
> * Jonathan Liu <net147@gmail.com> [150921 13:08]:
>
>> This avoids a "Could not unlink" warning when extracting a /var/run
>> symbolic link pointing to /run from the base-files package as it is
>> unable to unlink the /var/run directory when it contains opkg.lock.
>> This also fixes an issue where /var/run is created as a directory
>> instead of a symbolic link to /run.
> This fixes my recently discoverd runtime issues, when forwardporting a
> systemd-based distro from daisy/dizzy to the current master.
>
>> Signed-off-by: Jonathan Liu <net147@gmail.com>
> Acked-by: Anders Darander <anders@chargestorm.se>
>
>
Bump.
Regards,
Jonathan
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] opkg: create opkg.lock in /run instead of /var/run
2015-09-25 15:06 ` Jonathan Liu
@ 2015-09-28 6:42 ` Anders Darander
0 siblings, 0 replies; 6+ messages in thread
From: Anders Darander @ 2015-09-28 6:42 UTC (permalink / raw)
To: Jonathan Liu; +Cc: openembedded-core
* Jonathan Liu <net147@gmail.com> [150925 17:07]:
> On 21/09/2015 10:16 PM, Anders Darander wrote:
> >* Jonathan Liu <net147@gmail.com> [150921 13:08]:
> >>This avoids a "Could not unlink" warning when extracting a /var/run
> >>symbolic link pointing to /run from the base-files package as it is
> >>unable to unlink the /var/run directory when it contains opkg.lock.
> >>This also fixes an issue where /var/run is created as a directory
> >>instead of a symbolic link to /run.
> >This fixes my recently discoverd runtime issues, when forwardporting a
> >systemd-based distro from daisy/dizzy to the current master.
> >>Signed-off-by: Jonathan Liu <net147@gmail.com>
> >Acked-by: Anders Darander <anders@chargestorm.se>
> Bump.
This one needs a rebase. Could you send one?
Cheers,
Anders
--
Anders Darander
ChargeStorm AB / eStorm AB
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2015-09-28 6:42 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-09-21 11:09 [PATCH] opkg: create opkg.lock in /run instead of /var/run Jonathan Liu
2015-09-21 12:02 ` Jussi Kukkonen
2015-09-21 12:06 ` Jonathan Liu
2015-09-21 12:16 ` Anders Darander
2015-09-25 15:06 ` Jonathan Liu
2015-09-28 6:42 ` Anders Darander
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox