qemu looks for smbd at /usr/sbin

Done

Details

4 participants

Leo Famulari
Ludovic Courtès
Marius Bakke
Ricardo Wurmus

Owner: unassigned

Submitted by: Ricardo Wurmus

Severity: normal

Ricardo Wurmus wrote on 6 May 2017 12:26

Recipients:(address . bug-guix@gnu.org)

Message-ID:87zieq2sdx.fsf@elephly.net

Qemu comes with support for sharing files with a guest via Samba. This

is done by appending something like this to the qemu-system-*

invocation:

… -net user,smb=$HOME/Downloads -net nic

Qemu needs to be told *at configure time* where the smbd executable will

be at runtime. By default this is /usr/sbin/smbd. Users can work

around this problem by installing the “samba” package and creating a

link from /usr/sbin/smbd to the actual executable.

It would be better to fix this, but this would require adding samba as a

dependency to Qemu, so that the location can be set at configure time.

Alternatively, we could patch Qemu to look for smbd depending on an

environment variable.

Ricardo

GPG: BCA6 89B6 3655 3801 C3C6 2150 197A 5888 235F ACAC

https://elephly.net

Ludovic Courtès wrote on 6 May 2017 14:45

Recipients:(name . Ricardo Wurmus)(address . rekado@elephly.net)(address . 26797@debbugs.gnu.org)

Message-ID:878tmaw3us.fsf@gnu.org

Ricardo Wurmus <rekado@elephly.net> skribis:

Toggle quote (16 lines)> Qemu comes with support for sharing files with a guest via Samba.  This
> is done by appending something like this to the qemu-system-*
> invocation:
>
>     … -net user,smb=$HOME/Downloads -net nic
>
> Qemu needs to be told *at configure time* where the smbd executable will
> be at runtime.  By default this is /usr/sbin/smbd.  Users can work
> around this problem by installing the “samba” package and creating a
> link from /usr/sbin/smbd to the actual executable.
>
> It would be better to fix this, but this would require adding samba as a
> dependency to Qemu, so that the location can be set at configure time.
> Alternatively, we could patch Qemu to look for smbd depending on an
> environment variable.

See commit b344c505f4dff2ecbe981f0a0a3c3d67b222dcca, which removed the

dependency and Samba. Prior to that commit, QEMU recorded the absolute

file name of ‘smbd’.

The problem is that Samba is big:

Toggle snippet (6 lines)

$ guix size qemu |tail -1

total: 710.5 MiB

$ guix size qemu samba |tail -1

total: 1203.5 MiB

Thus it’d be best if we modified QEMU to search for ‘smbd’ in $PATH.

WDYT?

Ludo’.

Ricardo Wurmus wrote on 6 May 2017 15:03

Recipients:(name . Ludovic Courtès)(address . ludo@gnu.org)(address . 26797@debbugs.gnu.org)

Message-ID:87y3ua2l3c.fsf@elephly.net

Ludovic Courtès <ludo@gnu.org> writes:

Toggle quote (13 lines)> See commit b344c505f4dff2ecbe981f0a0a3c3d67b222dcca, which removed the
> dependency and Samba.  Prior to that commit, QEMU recorded the absolute
> file name of ‘smbd’.
>
> The problem is that Samba is big:
>
> --8<---------------cut here---------------start------------->8---
> $ guix size qemu |tail -1
> total: 710.5 MiB
> $ guix size qemu samba |tail -1
> total: 1203.5 MiB
> --8<---------------cut here---------------end--------------->8---

Right, that’s not good.

Toggle quote (2 lines)

> Thus it’d be best if we modified QEMU to search for ‘smbd’ in $PATH.

I agree! I can’t work on this right now, unfortunately, but I’ll

revisit this at some point in the future if nobody else beats me to it.

Thanks for your input!

Ricardo

GPG: BCA6 89B6 3655 3801 C3C6 2150 197A 5888 235F ACAC

https://elephly.net

Marius Bakke wrote on 6 May 2017 16:42

Recipients:(address . 26797@debbugs.gnu.org)

Message-ID:87y3uayrkc.fsf@fastmail.com

Ludovic Courtès <ludo@gnu.org> writes:

Toggle quote (33 lines)> Ricardo Wurmus <rekado@elephly.net> skribis:
>
>> Qemu comes with support for sharing files with a guest via Samba.  This
>> is done by appending something like this to the qemu-system-*
>> invocation:
>>
>>     … -net user,smb=$HOME/Downloads -net nic
>>
>> Qemu needs to be told *at configure time* where the smbd executable will
>> be at runtime.  By default this is /usr/sbin/smbd.  Users can work
>> around this problem by installing the “samba” package and creating a
>> link from /usr/sbin/smbd to the actual executable.
>>
>> It would be better to fix this, but this would require adding samba as a
>> dependency to Qemu, so that the location can be set at configure time.
>> Alternatively, we could patch Qemu to look for smbd depending on an
>> environment variable.
>
> See commit b344c505f4dff2ecbe981f0a0a3c3d67b222dcca, which removed the
> dependency and Samba.  Prior to that commit, QEMU recorded the absolute
> file name of ‘smbd’.
>
> The problem is that Samba is big:
>
> --8<---------------cut here---------------start------------->8---
> $ guix size qemu |tail -1
> total: 710.5 MiB
> $ guix size qemu samba |tail -1
> total: 1203.5 MiB
> --8<---------------cut here---------------end--------------->8---
>
> Thus it’d be best if we modified QEMU to search for ‘smbd’ in $PATH.

Tangentially, would it make sense to add a "qemu-full" package? I'd like
to add RBD support when Ceph is stable on Hydra, but that adds another
~300MiB to the closure (and a very expensive dependency..).

Leo Famulari wrote on 6 May 2017 20:48

Recipients:(name . Marius Bakke)(address . mbakke@fastmail.com)

Message-ID:20170506184850.GA22485@jasmine

On Sat, May 06, 2017 at 04:42:59PM +0200, Marius Bakke wrote:

Toggle quote (39 lines)> Ludovic Courtès <ludo@gnu.org> writes:
> 
> > Ricardo Wurmus <rekado@elephly.net> skribis:
> >
> >> Qemu comes with support for sharing files with a guest via Samba.  This
> >> is done by appending something like this to the qemu-system-*
> >> invocation:
> >>
> >>     … -net user,smb=$HOME/Downloads -net nic
> >>
> >> Qemu needs to be told *at configure time* where the smbd executable will
> >> be at runtime.  By default this is /usr/sbin/smbd.  Users can work
> >> around this problem by installing the “samba” package and creating a
> >> link from /usr/sbin/smbd to the actual executable.
> >>
> >> It would be better to fix this, but this would require adding samba as a
> >> dependency to Qemu, so that the location can be set at configure time.
> >> Alternatively, we could patch Qemu to look for smbd depending on an
> >> environment variable.
> >
> > See commit b344c505f4dff2ecbe981f0a0a3c3d67b222dcca, which removed the
> > dependency and Samba.  Prior to that commit, QEMU recorded the absolute
> > file name of ‘smbd’.
> >
> > The problem is that Samba is big:
> >
> > --8<---------------cut here---------------start------------->8---
> > $ guix size qemu |tail -1
> > total: 710.5 MiB
> > $ guix size qemu samba |tail -1
> > total: 1203.5 MiB
> > --8<---------------cut here---------------end--------------->8---
> >
> > Thus it’d be best if we modified QEMU to search for ‘smbd’ in $PATH.
> 
> Tangentially, would it make sense to add a "qemu-full" package? I'd like
> to add RBD support when Ceph is stable on Hydra, but that adds another
> ~300MiB to the closure (and a very expensive dependency..).

If qemu-full is something that would be generically useful, sure.

Otherwise, maybe you could bring it to the potluck!

Ludovic Courtès wrote on 7 May 2017 17:33

Recipients:(name . Marius Bakke)(address . mbakke@fastmail.com)

Message-ID:87k25sptq3.fsf@gnu.org

Marius Bakke <mbakke@fastmail.com> skribis:

Toggle quote (39 lines)> Ludovic Courtès <ludo@gnu.org> writes:
>
>> Ricardo Wurmus <rekado@elephly.net> skribis:
>>
>>> Qemu comes with support for sharing files with a guest via Samba.  This
>>> is done by appending something like this to the qemu-system-*
>>> invocation:
>>>
>>>     … -net user,smb=$HOME/Downloads -net nic
>>>
>>> Qemu needs to be told *at configure time* where the smbd executable will
>>> be at runtime.  By default this is /usr/sbin/smbd.  Users can work
>>> around this problem by installing the “samba” package and creating a
>>> link from /usr/sbin/smbd to the actual executable.
>>>
>>> It would be better to fix this, but this would require adding samba as a
>>> dependency to Qemu, so that the location can be set at configure time.
>>> Alternatively, we could patch Qemu to look for smbd depending on an
>>> environment variable.
>>
>> See commit b344c505f4dff2ecbe981f0a0a3c3d67b222dcca, which removed the
>> dependency and Samba.  Prior to that commit, QEMU recorded the absolute
>> file name of ‘smbd’.
>>
>> The problem is that Samba is big:
>>
>> --8<---------------cut here---------------start------------->8---
>> $ guix size qemu |tail -1
>> total: 710.5 MiB
>> $ guix size qemu samba |tail -1
>> total: 1203.5 MiB
>> --8<---------------cut here---------------end--------------->8---
>>
>> Thus it’d be best if we modified QEMU to search for ‘smbd’ in $PATH.
>
> Tangentially, would it make sense to add a "qemu-full" package? I'd like
> to add RBD support when Ceph is stable on Hydra, but that adds another
> ~300MiB to the closure (and a very expensive dependency..).

The day after someone will suggest adding “qemu-fuller”. ;-)

Seriously yes, why not; just wondering whether it should be called

“qemu-ceph-enabled” or something along these lines.

Ludo’.

Ludovic Courtès wrote on 8 May 2017 16:30

control message for bug #26797

Recipients:(address . control@debbugs.gnu.org)

Message-ID:87zienl8uc.fsf@gnu.org

tags 26797 easy

Marius Bakke wrote on 7 Apr 2020 21:22

Re: bug#26797: qemu looks for smbd at /usr/sbin

Recipients:

Message-ID:87a73nm77r.fsf@devup.no

Ricardo Wurmus <rekado@elephly.net> writes:

Toggle quote (11 lines)> Qemu comes with support for sharing files with a guest via Samba.  This
> is done by appending something like this to the qemu-system-*
> invocation:
>
>     … -net user,smb=$HOME/Downloads -net nic
>
> Qemu needs to be told *at configure time* where the smbd executable will
> be at runtime.  By default this is /usr/sbin/smbd.  Users can work
> around this problem by installing the “samba” package and creating a
> link from /usr/sbin/smbd to the actual executable.

This was fixed by Rutger back in
84276503059d46c0a42240f8770f59090614d863 and
0e5d0f66b56221b7de49ab28b607633e413c9110.

Closed

Your comment

This issue is archived.

To comment on this conversation send an email to 26797@debbugs.gnu.org

To respond to this issue using the mumi CLI, first switch to it

mumi current 26797

Then, you may apply the latest patchset in this issue (with sign off)

mumi am -- -s

Or, compose a reply to this issue

mumi compose

Or, send patches to this issue

mumi send-email *.patch

is:open	open issues
is:done	closed issues
submitter:<who>	search issue submitter
author:<who>	search by message author
date:yesterday..now	search by issue date
mdate:3m..2d	search by message date