February 2009 - Wine-Devel - list.winehq.org

Re: ntdll: add a warning about running wine as root (resend)

by Alexandre Julliard

Vitaliy Margolen <wine-devel(a)kievinfo.com> writes: > Alexandre Julliard wrote: >> Again, please demonstrate the exact sequence that leads to an actual >> problem, not just vague hearsay of people reporting problems that may or >> may not have anything to do with this. >> > Will this work? BTW default Open SuSE configuration. > > vitaliy(a)dragon:~ $export WINEPREFIX=~/.wine-root > vitaliy(a)dragon:~ $sudo wine winecfg > root's password: > wine: created the configuration directory '/root/.wine' As you can see it's creating a new prefix under /root, so it's not messing up the user's prefix. That's perfectly fine. -- Alexandre Julliard julliard(a)winehq.org

Feb. 10, 2009

Re: [try2] dinput: joydevs[id].name (EVIOCGNAME) is product name, joydevs[id].device is a path

by Ben Klein

2009/2/10 Vincent Pelletier <plr.vincent(a)gmail.com>: > On Tue, Feb 10, 2009 at 8:51 AM, Ben Klein <shacklein(a)gmail.com> wrote: >> You're taking out the strcpy into device, and replacing it with a >> strcpy into instance_name. What value will the device field hold? > > Err... strcpy from instance_name to the other struct. It's strcpy(out, in). > It's wine which is providing the windows program with names. OK, disregard everything I said ... ever ;) Well, at least about the strcpy. Sorry, must be my dyslexia playing tricks on me again. Oh, and can you try to hit "Reply to all" so it goes back to the mailing list too?

Feb. 10, 2009

Re: dinput: joydevs[id].name (EVIOCGNAME) is product name, joydevs[id].device is a path

by Ben Klein

2009/2/10 Vincent Pelletier <plr.vincent(a)gmail.com>: > On Tue, Feb 10, 2009 at 1:00 AM, Ben Klein <shacklein(a)gmail.com> wrote: >> You're adding a new field to the JoyDev struct called instance_name, >> which if I read right will store the input device path, but what >> happens to the existing device field? It looks like it's just taking >> up space, not actually getting a value stored in it. > > There is a value stored in it: see the code just above the second > chunk in my patch. You're taking out the strcpy into device, and replacing it with a strcpy into instance_name. What value will the device field hold? > But maybe you mean that it might not be read any more... Which might > be the case, I didn't check. > > About using space, I was more "worried" about the duplication of data > between "name" (kernel-provided device string - not a path) and the > new "instance_name" (could not find a better name for that without > changing existing) which also contains the device string plus > "evdev%d" as Vitaliy proposed. When will the new "instance_name" field be accessed by Wine? Regarding Vitaliy's earlier post, it looks ike "name" (product name) and "instance_name" are the same on native Windows platform. If this is always true, returning "name" when asked for instance name would be valid (so, storing only one string). A counter-example of this would be useful. The other difference is that the evdev method returns the vendor as a part of the product name (hence "Logitech Logitech MOMO Racing" etc.) Should we be concerned about this?

Feb. 10, 2009

Re: ntdll: add a warning about running wine as root (resend)

by Kai Blin

On Monday 09 February 2009 17:29:19 IneedAname wrote: > On Mon, 9 Feb 2009 07:35:20 -0700 (GMT-07:00) > > James Mckenzie <jjmckenzie51(a)earthlink.net> wrote: > > Yes, there are legit reasons to run as root, one of them is the now > > famous ICMP 'Ping' unavailability issue in Linux. Experts know about > > this and how to work around it. Newbies, used to Windows and how it > > works, just figure that this is restricted to administrators, and switch > > to root to get around it. The proper command, as I have learned through > > years of UNIX administration, is 'su -' but newbies don't know this and > > most just use 'su'. > > Can you point me at a guide for using ICMP 'Ping' without root? > Sorry for being a noob. You want to look at POSIX capabilities. I keep forgetting which one gives access to raw sockets. Google will know. Cheers, Kai -- Kai Blin WorldForge developer http://www.worldforge.org/ Wine developer http://wiki.winehq.org/KaiBlin Samba team member http://www.samba.org/samba/team/ -- Will code for cotton.

Feb. 10, 2009

Re: ntdll: add a warning about running wine as root (resend)

by Vitaliy Margolen

Ben Klein wrote: > 2009/2/10 Vitaliy Margolen <wine-devel(a)kievinfo.com>: >> Alexandre Julliard wrote: >>> Again, please demonstrate the exact sequence that leads to an actual >>> problem, not just vague hearsay of people reporting problems that may or >>> may not have anything to do with this. >>> >> Will this work? BTW default Open SuSE configuration. >> >> vitaliy(a)dragon:~ $export WINEPREFIX=~/.wine-root >> vitaliy(a)dragon:~ $sudo wine winecfg >> root's password: > Since when did sudo ask for root's password? It (normally?) asks for > the user's password. > >> wine: created the configuration directory '/root/.wine' > Is WINEPREFIX being passed to wine via sudo? For that matter, is sudo > overriding $HOME on your system? Or have you run "su -c winecfg" and > just replaced it with "sudo" above? > >> Application tried to create a window, but no driver could be loaded. >> Make sure that your X server is running and that $DISPLAY is set correctly. > Interesting. xauth problem. Again, I'd be surprised if this happened > with sudo. Not so surprised if it happened with "su -". Quoting myself: >> default Open SuSE configuration. Here is all the not commented content of the /etc/sudoers: Defaults always_set_home Defaults env_reset # And a long list of LC_* allowed env vars: Defaults env_keep = "LANG LC_* ... LINGUAS XDG_SESSION_COOKIE" Defaults targetpw # ask for the password of the target user i.e. root ALL<--->ALL=(ALL) ALL # WARNING! Only use this together with 'Defaults targetpw'! root<-->ALL=(ALL) ALL Explains everything you see from 'sudo wine winecfg'. Vitaliy.

Feb. 10, 2009

Re: ntdll: add a warning about running wine as root (resend)

by Ben Klein

2009/2/10 Vitaliy Margolen <wine-devel(a)kievinfo.com>: > Alexandre Julliard wrote: >> Again, please demonstrate the exact sequence that leads to an actual >> problem, not just vague hearsay of people reporting problems that may or >> may not have anything to do with this. >> > Will this work? BTW default Open SuSE configuration. > > vitaliy(a)dragon:~ $export WINEPREFIX=~/.wine-root > vitaliy(a)dragon:~ $sudo wine winecfg > root's password: Since when did sudo ask for root's password? It (normally?) asks for the user's password. > wine: created the configuration directory '/root/.wine' Is WINEPREFIX being passed to wine via sudo? For that matter, is sudo overriding $HOME on your system? Or have you run "su -c winecfg" and just replaced it with "sudo" above? > Application tried to create a window, but no driver could be loaded. > Make sure that your X server is running and that $DISPLAY is set correctly. Interesting. xauth problem. Again, I'd be surprised if this happened with sudo. Not so surprised if it happened with "su -".

Feb. 10, 2009

Re: Regression in 1.1.14 for Athena Visual Studio

by Dan Kegel

Thanks. I spent some time looking at a similar regression reported by Uwe. I hope to have a patch this weekend. On Mon, Feb 9, 2009 at 6:19 PM, Erich Hoover <ehoover(a)mines.edu> wrote: > In Wine 1.1.14 Athena Visual Studio fails to build Fortran applications and > returns with an error, doing a git bisect gives this result: > ---- > ebe4a9e321800fbe6d2004a640cd276a28d15c38 is first bad commit > commit ebe4a9e321800fbe6d2004a640cd276a28d15c38 > Author: Dan Kegel <dank(a)kegel.com> > Date: Sun Jan 18 19:19:46 2009 -0800 > > msvcrt: Remove CRs earlier in ascii mode. > > :040000 040000 44cfaec7cd552af4c92f220ad34a06d864eff79d > 685d15aa69a9e0f91e0d4cbc933191a29837dce9 M dlls > ---- > > However, removing this patch from the latest git doesn't do enough - Athena > instead hangs indefinitely trying to build. I guessed that it would also be > necessary to remove the related patch "msvcrt: Remove CRs earlier in ascii > mode in fseek, too.": > http://source.winehq.org/git/wine.git/?a=blob;f=dlls/msvcrt/file.c;hb=7f3c7… > > Fortunately, that did the trick and got things working again. I don't have > a lot of free time right now, but if there's some information I can give > that will help in fixing this issue then please let me know and I will do my > best to provide it. A 15 day trial of Athena is available, this error can > be reproduced by attempting to build any sample in the "Curve Fitting > Models" folder. >

Feb. 10, 2009

Regression in 1.1.14 for Athena Visual Studio

by Erich Hoover

In Wine 1.1.14 Athena Visual Studio fails to build Fortran applications and returns with an error, doing a git bisect gives this result: ---- ebe4a9e321800fbe6d2004a640cd276a28d15c38 is first bad commit commit ebe4a9e321800fbe6d2004a640cd276a28d15c38 Author: Dan Kegel <dank(a)kegel.com> Date: Sun Jan 18 19:19:46 2009 -0800 msvcrt: Remove CRs earlier in ascii mode. :040000 040000 44cfaec7cd552af4c92f220ad34a06d864eff79d 685d15aa69a9e0f91e0d4cbc933191a29837dce9 M dlls ---- However, removing this patch from the latest git doesn't do enough - Athena instead hangs indefinitely trying to build. I guessed that it would also be necessary to remove the related patch "msvcrt: Remove CRs earlier in ascii mode in fseek, too.": http://source.winehq.org/git/wine.git/?a=blob;f=dlls/msvcrt/file.c;hb=7f3c7… Fortunately, that did the trick and got things working again. I don't have a lot of free time right now, but if there's some information I can give that will help in fixing this issue then please let me know and I will do my best to provide it. A 15 day trial of Athena is available, this error can be reproduced by attempting to build any sample in the "Curve Fitting Models" folder.

Feb. 10, 2009

Re: ntdll: add a warning about running wine as root (resend)

by Vitaliy Margolen

Austin English wrote: > On Mon, Feb 9, 2009 at 9:58 AM, James Mckenzie > <jjmckenzie51(a)earthlink.net> wrote: >> Aurimas Fišeras <aurimas(a)gmail.com> wrote: >>> Sent: Feb 9, 2009 7:45 AM >>> To: James Mckenzie <jjmckenzie51(a)earthlink.net> >>> Cc: wine-devel(a)winehq.org >>> Subject: Re: ntdll: add a warning about running wine as root (resend) >>> >>> James Mckenzie wrote: >>>> Alexandre Julliard <julliard(a)winehq.org> wrote: >>>> [...] >>>> New wine installation: >>>> >>>> su (no dash so root's environment is not picked up) >>>> wine notepad >>>> install various programs and use them. >>>> exit >>>> >>>> User logs in a second time after learning how to properly use Wine. >>>> Attempt to do anything with Wine in user space. Cannot do due to permissions problems. >>>> >>>> The solution: >>>> >>>> sudo rm -rf .wine >>> No, the solution is >>> sudo chown -R user:user .wine >> Correct. However, this is not being given as the solution. >> >> Also, if the user is on an alternate wine prefix this would change to: >> >> sudo chown -R <username>:users $WINEPREFIX >> >> Is this in the FAQ somewhere? >> >> James McKenzie >> >> > It is now (before, just said sudo rm -rf ~/.wine). > You forgetting all the files created under ~/Desktop, ~/.config and ~/.local. Those will be a bigger pita and source for lots of issues. Vitaliy.

Feb. 10, 2009

Re: ntdll: add a warning about running wine as root (resend)

by Vitaliy Margolen

Alexandre Julliard wrote: > James Mckenzie <jjmckenzie51(a)earthlink.net> writes: > >> It does not hurt anything but Wine's reputation. I don't know, but >> the phrase "I would rather be run over rather than run down" does >> apply here. In other words, we should be getting positive responses >> to user experiences, rather than dealing with newbies having to delete >> their .wine directories because they ran Wine as root without setting >> to the root environment. > > Again, please demonstrate the exact sequence that leads to an actual > problem, not just vague hearsay of people reporting problems that may or > may not have anything to do with this. > Will this work? BTW default Open SuSE configuration. vitaliy(a)dragon:~ $export WINEPREFIX=~/.wine-root vitaliy(a)dragon:~ $sudo wine winecfg root's password: wine: created the configuration directory '/root/.wine' Application tried to create a window, but no driver could be loaded. Make sure that your X server is running and that $DISPLAY is set correctly. Application tried to create a window, but no driver could be loaded. Make sure that your X server is running and that $DISPLAY is set correctly. err:ole:apartment_createwindowifneeded CreateWindow failed with error 1114 err:ole:apartment_createwindowifneeded CreateWindow failed with error 1114 Application tried to create a window, but no driver could be loaded. Make sure that your X server is running and that $DISPLAY is set correctly. err:ole:apartment_createwindowifneeded CreateWindow failed with error 1114 err:ole:apartment_createwindowifneeded CreateWindow failed with error -536870654 err:ole:apartment_createwindowifneeded CreateWindow failed with error -536870654 err:ole:apartment_createwindowifneeded CreateWindow failed with error -536870654 Could not load Mozilla. HTML rendering will be disabled. err:ole:apartment_createwindowifneeded CreateWindow failed with error -536870654 err:ole:apartment_createwindowifneeded CreateWindow failed with error -536870654 err:ole:apartment_createwindowifneeded CreateWindow failed with error 0 err:ole:apartment_createwindowifneeded CreateWindow failed with error -536870654 err:ole:apartment_createwindowifneeded CreateWindow failed with error 0 err:ole:apartment_createwindowifneeded CreateWindow failed with error -536870654 err:ole:apartment_createwindowifneeded CreateWindow failed with error -536870654 err:wgl:process_attach X11DRV or GDI32 not loaded. Cannot create default context. wine: configuration in '/root/.wine' has been updated. Application tried to create a window, but no driver could be loaded. Make sure that your X server is running and that $DISPLAY is set correctly. Application tried to create a window, but no driver could be loaded. Make sure that your X server is running and that $DISPLAY is set correctly. err:ole:apartment_createwindowifneeded CreateWindow failed with error 1114

Feb. 10, 2009