June 2009 - Wine-devel - list.winehq.org

Re: Alexandre Julliard : configure: Detect the appropriate form for the __ASM_GLOBAL_FUNC macro.
by Steven Edwards 11 Jun '09

11 Jun '09

Hi, On Thu, Jun 11, 2009 at 11:54 AM, Alexandre Julliard<julliard(a)winehq.org> wrote: > Module: wine > Branch: master > Commit: 857f1e0924f23865038a608ee6834ef524371f20 > URL: http://source.winehq.org/git/wine.git/?a=commit;h=857f1e0924f23865038a608ee… > > Author: Alexandre Julliard <julliard(a)winehq.org> > Date: Thu Jun 11 16:32:42 2009 +0200 > > configure: Detect the appropriate form for the __ASM_GLOBAL_FUNC macro. I believe this patch is causing a problem for me on OS X Leopard gcc version 4.0.1 (Apple Inc. build 5465). When I try to compile riched20 with a clean tree at HEAD I am getting this ld: absolute addressing (perhaps -mdynamic-no-pic) used in _itextHostStdcallVtbl from txthost.o not allowed in slidable image collect2: ld returned 1 exit status winegcc: gcc failed make: *** [riched20.dll.so] Error 2 Is the __ASM_GLOBAL_FUNC stdcall wrapper in txthost.c just borked? Thanks -- Steven Edwards "There is one thing stronger than all the armies in the world, and that is an idea whose time has come." - Victor Hugo

1 0

Re: RDo we still need the regression test for 1.1.23 ?
by Ben Klein 11 Jun '09

11 Jun '09

2009/6/12 chris ahrendt <celticht32(a)yahoo.com>: > > Ben Klein wrote: >> 2009/6/11 chris ahrendt <celticht32(a)yahoo.com>: >> >>> to see what patch caused the failure on non NVIDIA cards? >>> >> >> Where's the rest of your message? Oh, I see, it's in the subject. >> >> Wasn't there already a thread about this? >> >> Back on topic, if it is definitely a regression then a regression test >> on bugzilla will be useful. Or at least, some information on how to >> reproduce it. >> >> >> > The test is easy... use a ATI card... and run 1.1.23 with or without > the Registry value to set it back to the old way of rendering. Not so easy if you don't have an ATI card, or if you don't know what apps cause the error. Wine Notepad seems to be working fine for me, no crashes.

1 0

RDo we still need the regression test for 1.1.23 ?
by chris ahrendt 11 Jun '09

11 Jun '09

to see what patch caused the failure on non NVIDIA cards? Chris

2 1

Fwd: RDo we still need the regression test for 1.1.23 ?
by Danila Sentiabov 11 Jun '09

11 Jun '09

Forwarding to mailing list... ---------- Forwarded message ---------- From: Danila Sentiabov <dsent.zen(a)gmail.com> Date: Thu, Jun 11, 2009 at 23:02 Subject: Re: RDo we still need the regression test for 1.1.23 ? To: chris ahrendt <celticht32(a)yahoo.com> I didn't follow the topic, but as far as I know, if there is a regression it's surely begging for regression test. On which application the failure can be reproduced? On Wed, Jun 10, 2009 at 22:47, chris ahrendt <celticht32(a)yahoo.com> wrote: > > to see what patch caused the failure on non NVIDIA cards? > > > Chris > -- Best regards, Danila Sentiabov aka dsent -- Best regards, Danila Sentiabov aka dsent

1 0

Re: [Bugzilla]: Refuse to accept comments with big number of logs / back traces
by Vitaliy Margolen 11 Jun '09

11 Jun '09

Vitaliy Margolen wrote: > The limits are: > 20 lines for logs (fixme, trace, err, warn) > 10 lines for back traces > --- > Bugzilla/Bug.pm | 7 +++++++ > template/en/default/global/user-error.html.tmpl | 4 ++++ > 2 files changed, 11 insertions(+), 0 deletions(-) > So who is the one to commit this? Alexandre? Jeremy? Vitaliy.

5 7

Re: winemenubuilder: generate new menus in the right directory
by Alexandre Julliard 11 Jun '09

11 Jun '09

Damjan Jovanovic <damjan.jov(a)gmail.com> writes: > @@ -882,7 +882,11 @@ static BOOL write_menu_file(const char *filename) > > while (1) > { > + char *tmpdir = getenv("TMPDIR"); > + unsetenv("TMPDIR"); > tempfilename = tempnam(xdg_config_dir, "_wine"); > + if (tmpdir) > + setenv("TMPDIR", tmpdir, 1); That's ugly. You should use mkstemps() or something like that. -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: Fix incorrect check in WCMD_HandleTildaModifiers().
by Alexandre Julliard 11 Jun '09

11 Jun '09

Gerald Pfeifer <gerald(a)pfeifer.com> writes: >>= '0' || <= '9' always is true. :-) This patch fixes it, and > avoids two just hand coded checks altogether. > > Gerald > > ChangeLog: > Fix incorrect check in WCMD_HandleTildaModifiers(). > Use isdigit() instead of hand-coding this check. Using isdigit() on a Unicode char is not a good idea (and in this case isdigitW() wouldn't be a good idea either). -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: opengl32: get rid of glext.h
by Detlef Riekenberg 11 Jun '09

11 Jun '09

On Mi, 2009-06-10 at 21:56 +0200, Roderick Colenbrander wrote: > -#ifdef HAVE_GL_GLEXT_H > -# include <GL/glext.h> > -#endif The same code is used in winex11.drv/opengl.c Can this also be removed? When that's the case, then "GL/glext.h" can be removed from AC_CHECK_HEADERS in configure.ac -- By by ... Detlef

2 1

Re: [2/5] WineD3D: Implement loops with the NV extensions
by Allan Tong 10 Jun '09

10 Jun '09

> + if(vshader) > + { > + struct shader_arb_ctx_priv *priv = ins->ctx->backend_data; > + struct list *e = list_head(&priv->control_frames); > + struct control_frame *control_frame = LIST_ENTRY(e, struct control_frame, entry); > + > + if(priv->loop_depth > 1) shader_addline(buffer, "PUSHA aL;\n"); > + > + shader_addline(buffer, "ARLC aL, %s.xywz;\n", src_name); > + shader_addline(buffer, "BRA loop_%u_end (LE.x);\n", control_frame->loop_no); > + shader_addline(buffer, "loop_%u_start:\n", control_frame->loop_no); > + } Nested loops still don't work here. You need NV_vertex_program3 in order to use PUSHA and POPA. - Allan

3 3

Re: opengl32: get rid of glext.h
by Austin English 10 Jun '09

10 Jun '09

On Wed, Jun 10, 2009 at 2:56 PM, Roderick Colenbrander<thunderbird2k(a)gmail.com> wrote: > Hi, > > Since opengl32 doesn't export opengl 1.2 functions anymore from the > spec file we don't need glext.h anymore. This also helps compilation > on OSX. > > Roderick Works great. Thanks for the quick work Roderick! -- -Austin

1 0