Opened 16 years ago

Closed 16 years ago

Last modified 4 years ago

#3488 closed enhancement (junk)

NetSurf -- new Haiku native build available

Reported by: mmadia Owned by: axeld
Priority: normal Milestone:
Component: Applications Version: R1/pre-alpha1
Keywords: Cc:
Blocked By: Blocking:
Platform: All

Description

A replacement build of NetSurf is available at : package: http://bezilla.beuser.de/builds/NetSurf-haiku-gcc2.zip

md5sum: 3d0fd8ba49dcde4708a70947e75a900e NetSurf-haiku-gcc2.zip

The first link I provided in #haiku, was not archived properly and needed to be extracted to a temporary folder and then manually installed.

This new link, which uses the same filename can be expanded directly to /boot

This build extracts it's 3rd party libs to /boot/common The only testing I have done is to check for #3449. Other than that, I have not thoroughly tested and compared this binary to the previous (and current) version available in OptionalPackages.

Could someone do this testing? Then based on those results, upload the binary to haiku-files and update OptionalPackages

Attachments (2)

db.jpg (222.0 KB ) - added by kvdman 16 years ago.
netsurf-haiku-bt.txt (7.6 KB ) - added by mmadia 16 years ago.
complete backtrace of error.

Download all attachments as: .zip

Change History (12)

comment:1 by mmadia, 16 years ago

Also, BeHappy's functionality was not tested.

by kvdman, 16 years ago

Attachment: db.jpg added

comment:2 by kvdman, 16 years ago

Hi,

I've tested the binary. Seems to run out of memory when executed, see the attached.

comment:3 by mmadia, 16 years ago

thanks karl.

Here's a new build. So far, that out-of-memory error hasn't popped up.

http://bezilla.beuser.de/builds/NetSurf-r6641-haiku-gcc2.zip

md5sum: 01d28cd34366c766bb54c7e10d4bbd9a

comment:4 by mmadia, 16 years ago

For some reason, I'm not experiencing this issue on the first run on a clean system. Subsequent runs exhibit it.

Putting this on pause for the time being.

by mmadia, 16 years ago

Attachment: netsurf-haiku-bt.txt added

complete backtrace of error.

comment:5 by mmadia, 16 years ago

I'm curious if this is triggered from dropped network data. Clicking [OK], instead of [Debug] allows me to attempt re-loading the page. Eventually it'll load properly without generating this error.

This section of code is where the error actually starts.

The line with "*" is line 344

render/html.c::html_convert

	if (c->data.html.encoding == NULL) {
		const char *encoding = binding_get_encoding(
				c->data.html.parser_binding, 
				&c->data.html.encoding_source);

		c->data.html.encoding = talloc_strdup(c, encoding);
		if (c->data.html.encoding == NULL) {
			msg_data.error = messages_get("NoMemory");
***			content_broadcast(c, CONTENT_MSG_ERROR, msg_data);
			return false;
		}
	}

comment:6 by umccullough, 16 years ago

It occurred to me that these warnings are *not* supposed to land a user in GDB..

So looking here:

http://source.netsurf-browser.org/trunk/netsurf/beos/beos_gui.cpp?view=markup

I see that the warn_user() function has some conditional code to either show an alert, or also die in the debugger - I'm guessing the currently enabled code is not preferable - and the #if 0 should be switched around...

also same for die() - looks like this was debugging code left in.

comment:7 by mmadia, 16 years ago

I discussed this with Urias in IRC and forgot to update the ticket.

These crashes aren't actual crashes, but rather are similar to an alert dialog that gives the user an *option* to open GDB.

If I click [OK], then I can attempt to reload the page. Eventually the page will load This adds weight to my belief that the above issue is a direct result of dropped network data. Why this doesn't occur in the current built-in-bone package, I don't know. Perhaps this mechanism was disabled or simply not implemented at that svn revision?

comment:8 by stippi, 16 years ago

Since Francois fixed the BeOS NetSurf build in SVN, I've been able to build NetSurf on Haiku. The issue with the error message about ./beos/res happens when the NetSurf resources are not installed to /boot/home/config/settings/NetSurf. In the source, look into beos/res, there are a bunch of links to resource files in other parts of the source tree. Just copy the originals with the same name as the link into the settings folder. Obviously, it also needs to be part of the package.

comment:9 by mmadia, 16 years ago

Resolution: junk
Status: newclosed

This build is no longer necessary. #3785 points to the new build, which was accepted in changeset:30240 closing as junk.

comment:10 by pulkomandy, 4 years ago

Milestone: R1
Note: See TracTickets for help on using tickets.