* Re: [Qemu-devel] [Bug 1817239] [NEW] add '--targets' option to qemu-binfmt-conf.sh
2019-02-22 3:23 [Qemu-devel] [Bug 1817239] [NEW] add '--targets' option to qemu-binfmt-conf.sh umarcor
@ 2019-02-22 7:32 ` Laurent Vivier
2019-03-09 1:35 ` [Qemu-devel] [Bug 1817239] " umarcor
` (3 subsequent siblings)
4 siblings, 0 replies; 6+ messages in thread
From: Laurent Vivier @ 2019-02-22 7:32 UTC (permalink / raw)
To: qemu-devel
On 22/02/2019 04:31, Launchpad Bug Tracker wrote:
> I'd like to ask for the addition of option '--targets' to scripts/qemu-
> binfmt-conf.sh, in order to allow registering the interpreters for the
> given list of architectures only, instead of using all of the ones
> defined in qemu_target_list. The following is a possible patch that
> implements it:
>
> qemu-binfmt-conf.sh | 9 ++++++++-
> 1 file changed, 8 insertions(+), 1 deletion(-)
Please send your patch to the qemu-devel mailing list.
I think it's a good idea but we should homogenize "--debian" and
"--systemd": remove the parameter from "--systemd" to generate by
default all the targets, and allow the user to provide a subset of the
targets using the newly introduced "--target" parameter.
Thanks,
Laurent
--
You received this bug notification because you are a member of qemu-
devel-ml, which is subscribed to QEMU.
https://bugs.launchpad.net/bugs/1817239
Title:
add '--targets' option to qemu-binfmt-conf.sh
Status in QEMU:
New
Bug description:
I'd like to ask for the addition of option '--targets' to scripts
/qemu-binfmt-conf.sh, in order to allow registering the interpreters
for the given list of architectures only, instead of using all of the
ones defined in qemu_target_list. The following is a possible patch
that implements it:
qemu-binfmt-conf.sh | 9 ++++++++-
1 file changed, 8 insertions(+), 1 deletion(-)
diff --git a/qemu-binfmt-conf.sh b/qemu-binfmt-conf.sh
index b5a1674..be4a19b 100644
--- a/qemu-binfmt-conf.sh
+++ b/qemu-binfmt-conf.sh
@@ -170,6 +170,7 @@ usage() {
Usage: qemu-binfmt-conf.sh [--qemu-path PATH][--debian][--systemd CPU]
[--help][--credential yes|no][--exportdir PATH]
[--persistent yes|no][--qemu-suffix SUFFIX]
+ [--targets TARGETS]
Configure binfmt_misc to use qemu interpreter
@@ -189,6 +190,8 @@ Usage: qemu-binfmt-conf.sh [--qemu-path PATH][--debian][--systemd CPU]
--persistent: if yes, the interpreter is loaded when binfmt is
configured and remains in memory. All future uses
are cloned from the open file.
+ --targets: comma-separated list of targets. If provided, only
+ the targets in the list are registered.
To import templates with update-binfmts, use :
@@ -324,7 +327,7 @@ CREDENTIAL=no
PERSISTENT=no
QEMU_SUFFIX=""
-options=$(getopt -o ds:Q:S:e:hc:p: -l debian,systemd:,qemu-path:,qemu-suffix:,exportdir:,help,credential:,persistent: -- "$@")
+options=$(getopt -o ds:Q:S:e:hc:p:t: -l debian,systemd:,qemu-path:,qemu-suffix:,exportdir:,help,credential:,persistent:,targets: -- "$@")
eval set -- "$options"
while true ; do
@@ -380,6 +383,10 @@ while true ; do
shift
PERSISTENT="$1"
;;
+ -t|--targets)
+ shift
+ qemu_target_list="$(echo "$1" | tr ',' ' ')"
+ ;;
*)
break
;;
--
2.20.1
To manage notifications about this bug go to:
https://bugs.launchpad.net/qemu/+bug/1817239/+subscriptions
^ permalink raw reply [flat|nested] 6+ messages in thread
* [Qemu-devel] [Bug 1817239] Re: add '--targets' option to qemu-binfmt-conf.sh
2019-02-22 3:23 [Qemu-devel] [Bug 1817239] [NEW] add '--targets' option to qemu-binfmt-conf.sh umarcor
2019-02-22 7:32 ` Laurent Vivier
@ 2019-03-09 1:35 ` umarcor
2020-11-04 23:04 ` John Snow
` (2 subsequent siblings)
4 siblings, 0 replies; 6+ messages in thread
From: umarcor @ 2019-03-09 1:35 UTC (permalink / raw)
To: qemu-devel
I submitted a first version some days ago, which homogeneized the
implementation, as suggested by Laurent Vivier. It received some
feedback from Eric Blake. A patchset (v3) is ready for review:
https://patchew.org/QEMU/20190306031221.GA53@03612eec87fc/#
The feature requested in this issue is included in the patchset:
[Qemu-devel] [PATCH v3 6/10] qemu-binfmt-conf.sh: generalize <CPU> to positional <CPUS>
https://patchew.org/QEMU/20190306031221.GA53@03612eec87fc/20190306045019.GF75@03612eec87fc/
Note that, instead of adding a parameter named `--target`, positional
arguments are used, which where not being processed at all.
** Changed in: qemu
Status: New => In Progress
--
You received this bug notification because you are a member of qemu-
devel-ml, which is subscribed to QEMU.
https://bugs.launchpad.net/bugs/1817239
Title:
add '--targets' option to qemu-binfmt-conf.sh
Status in QEMU:
In Progress
Bug description:
I'd like to ask for the addition of option '--targets' to scripts
/qemu-binfmt-conf.sh, in order to allow registering the interpreters
for the given list of architectures only, instead of using all of the
ones defined in qemu_target_list. The following is a possible patch
that implements it:
qemu-binfmt-conf.sh | 9 ++++++++-
1 file changed, 8 insertions(+), 1 deletion(-)
diff --git a/qemu-binfmt-conf.sh b/qemu-binfmt-conf.sh
index b5a1674..be4a19b 100644
--- a/qemu-binfmt-conf.sh
+++ b/qemu-binfmt-conf.sh
@@ -170,6 +170,7 @@ usage() {
Usage: qemu-binfmt-conf.sh [--qemu-path PATH][--debian][--systemd CPU]
[--help][--credential yes|no][--exportdir PATH]
[--persistent yes|no][--qemu-suffix SUFFIX]
+ [--targets TARGETS]
Configure binfmt_misc to use qemu interpreter
@@ -189,6 +190,8 @@ Usage: qemu-binfmt-conf.sh [--qemu-path PATH][--debian][--systemd CPU]
--persistent: if yes, the interpreter is loaded when binfmt is
configured and remains in memory. All future uses
are cloned from the open file.
+ --targets: comma-separated list of targets. If provided, only
+ the targets in the list are registered.
To import templates with update-binfmts, use :
@@ -324,7 +327,7 @@ CREDENTIAL=no
PERSISTENT=no
QEMU_SUFFIX=""
-options=$(getopt -o ds:Q:S:e:hc:p: -l debian,systemd:,qemu-path:,qemu-suffix:,exportdir:,help,credential:,persistent: -- "$@")
+options=$(getopt -o ds:Q:S:e:hc:p:t: -l debian,systemd:,qemu-path:,qemu-suffix:,exportdir:,help,credential:,persistent:,targets: -- "$@")
eval set -- "$options"
while true ; do
@@ -380,6 +383,10 @@ while true ; do
shift
PERSISTENT="$1"
;;
+ -t|--targets)
+ shift
+ qemu_target_list="$(echo "$1" | tr ',' ' ')"
+ ;;
*)
break
;;
--
2.20.1
To manage notifications about this bug go to:
https://bugs.launchpad.net/qemu/+bug/1817239/+subscriptions
^ permalink raw reply [flat|nested] 6+ messages in thread
* [Bug 1817239] Re: add '--targets' option to qemu-binfmt-conf.sh
2019-02-22 3:23 [Qemu-devel] [Bug 1817239] [NEW] add '--targets' option to qemu-binfmt-conf.sh umarcor
2019-02-22 7:32 ` Laurent Vivier
2019-03-09 1:35 ` [Qemu-devel] [Bug 1817239] " umarcor
@ 2020-11-04 23:04 ` John Snow
2021-04-20 8:21 ` Thomas Huth
2021-06-20 4:17 ` Launchpad Bug Tracker
4 siblings, 0 replies; 6+ messages in thread
From: John Snow @ 2020-11-04 23:04 UTC (permalink / raw)
To: qemu-devel
This series appears to have stalled as of v7:
https://lists.gnu.org/archive/html/qemu-devel/2019-03/msg04241.html --
moving back to 'New' status.
** Changed in: qemu
Status: In Progress => New
--
You received this bug notification because you are a member of qemu-
devel-ml, which is subscribed to QEMU.
https://bugs.launchpad.net/bugs/1817239
Title:
add '--targets' option to qemu-binfmt-conf.sh
Status in QEMU:
New
Bug description:
I'd like to ask for the addition of option '--targets' to scripts
/qemu-binfmt-conf.sh, in order to allow registering the interpreters
for the given list of architectures only, instead of using all of the
ones defined in qemu_target_list. The following is a possible patch
that implements it:
qemu-binfmt-conf.sh | 9 ++++++++-
1 file changed, 8 insertions(+), 1 deletion(-)
diff --git a/qemu-binfmt-conf.sh b/qemu-binfmt-conf.sh
index b5a1674..be4a19b 100644
--- a/qemu-binfmt-conf.sh
+++ b/qemu-binfmt-conf.sh
@@ -170,6 +170,7 @@ usage() {
Usage: qemu-binfmt-conf.sh [--qemu-path PATH][--debian][--systemd CPU]
[--help][--credential yes|no][--exportdir PATH]
[--persistent yes|no][--qemu-suffix SUFFIX]
+ [--targets TARGETS]
Configure binfmt_misc to use qemu interpreter
@@ -189,6 +190,8 @@ Usage: qemu-binfmt-conf.sh [--qemu-path PATH][--debian][--systemd CPU]
--persistent: if yes, the interpreter is loaded when binfmt is
configured and remains in memory. All future uses
are cloned from the open file.
+ --targets: comma-separated list of targets. If provided, only
+ the targets in the list are registered.
To import templates with update-binfmts, use :
@@ -324,7 +327,7 @@ CREDENTIAL=no
PERSISTENT=no
QEMU_SUFFIX=""
-options=$(getopt -o ds:Q:S:e:hc:p: -l debian,systemd:,qemu-path:,qemu-suffix:,exportdir:,help,credential:,persistent: -- "$@")
+options=$(getopt -o ds:Q:S:e:hc:p:t: -l debian,systemd:,qemu-path:,qemu-suffix:,exportdir:,help,credential:,persistent:,targets: -- "$@")
eval set -- "$options"
while true ; do
@@ -380,6 +383,10 @@ while true ; do
shift
PERSISTENT="$1"
;;
+ -t|--targets)
+ shift
+ qemu_target_list="$(echo "$1" | tr ',' ' ')"
+ ;;
*)
break
;;
--
2.20.1
To manage notifications about this bug go to:
https://bugs.launchpad.net/qemu/+bug/1817239/+subscriptions
^ permalink raw reply [flat|nested] 6+ messages in thread
* [Bug 1817239] Re: add '--targets' option to qemu-binfmt-conf.sh
2019-02-22 3:23 [Qemu-devel] [Bug 1817239] [NEW] add '--targets' option to qemu-binfmt-conf.sh umarcor
` (2 preceding siblings ...)
2020-11-04 23:04 ` John Snow
@ 2021-04-20 8:21 ` Thomas Huth
2021-06-20 4:17 ` Launchpad Bug Tracker
4 siblings, 0 replies; 6+ messages in thread
From: Thomas Huth @ 2021-04-20 8:21 UTC (permalink / raw)
To: qemu-devel
The QEMU project is currently considering to move its bug tracking to another system. For this we need to know which bugs are still valid and which could be closed already. Thus we are setting older bugs to "Incomplete" now.
If you still think this bug report here is valid, then please switch the state back to "New" within the next 60 days, otherwise this report will be marked as "Expired". Or mark it as "Fix Released" if the problem has been solved with a newer version of QEMU already. Thank you and sorry for the inconvenience.
** Changed in: qemu
Status: New => Incomplete
--
You received this bug notification because you are a member of qemu-
devel-ml, which is subscribed to QEMU.
https://bugs.launchpad.net/bugs/1817239
Title:
add '--targets' option to qemu-binfmt-conf.sh
Status in QEMU:
Incomplete
Bug description:
I'd like to ask for the addition of option '--targets' to scripts
/qemu-binfmt-conf.sh, in order to allow registering the interpreters
for the given list of architectures only, instead of using all of the
ones defined in qemu_target_list. The following is a possible patch
that implements it:
qemu-binfmt-conf.sh | 9 ++++++++-
1 file changed, 8 insertions(+), 1 deletion(-)
diff --git a/qemu-binfmt-conf.sh b/qemu-binfmt-conf.sh
index b5a1674..be4a19b 100644
--- a/qemu-binfmt-conf.sh
+++ b/qemu-binfmt-conf.sh
@@ -170,6 +170,7 @@ usage() {
Usage: qemu-binfmt-conf.sh [--qemu-path PATH][--debian][--systemd CPU]
[--help][--credential yes|no][--exportdir PATH]
[--persistent yes|no][--qemu-suffix SUFFIX]
+ [--targets TARGETS]
Configure binfmt_misc to use qemu interpreter
@@ -189,6 +190,8 @@ Usage: qemu-binfmt-conf.sh [--qemu-path PATH][--debian][--systemd CPU]
--persistent: if yes, the interpreter is loaded when binfmt is
configured and remains in memory. All future uses
are cloned from the open file.
+ --targets: comma-separated list of targets. If provided, only
+ the targets in the list are registered.
To import templates with update-binfmts, use :
@@ -324,7 +327,7 @@ CREDENTIAL=no
PERSISTENT=no
QEMU_SUFFIX=""
-options=$(getopt -o ds:Q:S:e:hc:p: -l debian,systemd:,qemu-path:,qemu-suffix:,exportdir:,help,credential:,persistent: -- "$@")
+options=$(getopt -o ds:Q:S:e:hc:p:t: -l debian,systemd:,qemu-path:,qemu-suffix:,exportdir:,help,credential:,persistent:,targets: -- "$@")
eval set -- "$options"
while true ; do
@@ -380,6 +383,10 @@ while true ; do
shift
PERSISTENT="$1"
;;
+ -t|--targets)
+ shift
+ qemu_target_list="$(echo "$1" | tr ',' ' ')"
+ ;;
*)
break
;;
--
2.20.1
To manage notifications about this bug go to:
https://bugs.launchpad.net/qemu/+bug/1817239/+subscriptions
^ permalink raw reply [flat|nested] 6+ messages in thread
* [Bug 1817239] Re: add '--targets' option to qemu-binfmt-conf.sh
2019-02-22 3:23 [Qemu-devel] [Bug 1817239] [NEW] add '--targets' option to qemu-binfmt-conf.sh umarcor
` (3 preceding siblings ...)
2021-04-20 8:21 ` Thomas Huth
@ 2021-06-20 4:17 ` Launchpad Bug Tracker
4 siblings, 0 replies; 6+ messages in thread
From: Launchpad Bug Tracker @ 2021-06-20 4:17 UTC (permalink / raw)
To: qemu-devel
[Expired for QEMU because there has been no activity for 60 days.]
** Changed in: qemu
Status: Incomplete => Expired
--
You received this bug notification because you are a member of qemu-
devel-ml, which is subscribed to QEMU.
https://bugs.launchpad.net/bugs/1817239
Title:
add '--targets' option to qemu-binfmt-conf.sh
Status in QEMU:
Expired
Bug description:
I'd like to ask for the addition of option '--targets' to scripts
/qemu-binfmt-conf.sh, in order to allow registering the interpreters
for the given list of architectures only, instead of using all of the
ones defined in qemu_target_list. The following is a possible patch
that implements it:
qemu-binfmt-conf.sh | 9 ++++++++-
1 file changed, 8 insertions(+), 1 deletion(-)
diff --git a/qemu-binfmt-conf.sh b/qemu-binfmt-conf.sh
index b5a1674..be4a19b 100644
--- a/qemu-binfmt-conf.sh
+++ b/qemu-binfmt-conf.sh
@@ -170,6 +170,7 @@ usage() {
Usage: qemu-binfmt-conf.sh [--qemu-path PATH][--debian][--systemd CPU]
[--help][--credential yes|no][--exportdir PATH]
[--persistent yes|no][--qemu-suffix SUFFIX]
+ [--targets TARGETS]
Configure binfmt_misc to use qemu interpreter
@@ -189,6 +190,8 @@ Usage: qemu-binfmt-conf.sh [--qemu-path PATH][--debian][--systemd CPU]
--persistent: if yes, the interpreter is loaded when binfmt is
configured and remains in memory. All future uses
are cloned from the open file.
+ --targets: comma-separated list of targets. If provided, only
+ the targets in the list are registered.
To import templates with update-binfmts, use :
@@ -324,7 +327,7 @@ CREDENTIAL=no
PERSISTENT=no
QEMU_SUFFIX=""
-options=$(getopt -o ds:Q:S:e:hc:p: -l debian,systemd:,qemu-path:,qemu-suffix:,exportdir:,help,credential:,persistent: -- "$@")
+options=$(getopt -o ds:Q:S:e:hc:p:t: -l debian,systemd:,qemu-path:,qemu-suffix:,exportdir:,help,credential:,persistent:,targets: -- "$@")
eval set -- "$options"
while true ; do
@@ -380,6 +383,10 @@ while true ; do
shift
PERSISTENT="$1"
;;
+ -t|--targets)
+ shift
+ qemu_target_list="$(echo "$1" | tr ',' ' ')"
+ ;;
*)
break
;;
--
2.20.1
To manage notifications about this bug go to:
https://bugs.launchpad.net/qemu/+bug/1817239/+subscriptions
^ permalink raw reply [flat|nested] 6+ messages in thread