Bug 80401 - links 2.1pre17 released + typo
Bug#: 80401 Product:  Gentoo Linux Version: unspecified Platform: All
OS/Version: Linux Status: RESOLVED Severity: enhancement Priority: P2
Resolution: FIXED Assigned To: seemant@gentoo.org Reported By: legion@birra.it
Component: Ebuilds
URL:  http://atrey.karlin.mff.cuni.cz/~clock/twibright/links/download.html
Summary: links 2.1pre17 released + typo
Keywords:  
Status Whiteboard: 
Opened: 2005-02-01 16:09 0000
Description:   Opened: 2005-02-01 16:09 0000
links 2.1pre16 released
and there is a typo in description: "links is a fast lightweight text tand graphic web-browser"

Reproducible: Always
Steps to Reproduce:

------- Comment #1 From Ingo -LaSombra- Hoffmann 2005-03-04 04:21:52 0000 -------
Created an attachment (id=52641) [details]
ebuild + fixed typo

------- Comment #2 From Ingo -LaSombra- Hoffmann 2005-03-04 04:22:25 0000 -------
Alessandro,

There's the new links version, take a look a report?
:)

------- Comment #3 From Alessandro Lepore 2005-03-04 08:36:47 0000 -------
seems ok, but USE=unicode doesn't work:
the patch doesn't applies (and also needs to be renamed to pre17)

------- Comment #4 From Ingo -LaSombra- Hoffmann 2005-03-07 04:29:14 0000 -------
Created an attachment (id=52843) [details]
Just to make it easy to upload to portage

------- Comment #5 From Ingo -LaSombra- Hoffmann 2005-03-07 04:29:43 0000 -------
Hope this hit portage soon
:)

------- Comment #6 From Seemant Kulleen (RETIRED) 2005-03-14 08:44:13 0000 -------
Ingo, I really prefer text files a attachments, to be honest.

Thanks :)

------- Comment #7 From Seemant Kulleen (RETIRED) 2005-03-15 12:44:43 0000 -------
Actually, the patch fails, so I need to fix0r it up to work with pre17

------- Comment #8 From Marcelo Goes 2005-03-19 17:50:03 0000 -------
Created an attachment (id=53901) [details]
links-2.1pre17-utf8.diff

I ported the current pre15 patch to pre17. It compiled and worked fine here
just renaming the current ebuild.

I would just like to call attention to terminal_options() in menu.c;
mem_alloc() was replaced with mem_calloc() and I'm not sure if just replacing
12 with 14 cuts it.

------- Comment #9 From Seemant Kulleen (RETIRED) 2005-03-22 08:09:17 0000 -------
marcelo I have no idea -- but what exactly do you mean by "replacing 12 with 14
cuts it"?  Where is the replacement, and what is it cutting?

------- Comment #10 From Marcelo Goes 2005-03-22 12:48:03 0000 -------
Seemant, using this:

-	d = mem_calloc(sizeof(struct dialog) + 12 * sizeof(struct dialog_item));
+	d = mem_calloc(sizeof(struct dialog) + 14 * sizeof(struct dialog_item));

instead of:

-	if (!(d = mem_alloc(sizeof(struct dialog) + 12 * sizeof(struct dialog_item)))) return;
-	memset(d, 0, sizeof(struct dialog) + 12 * sizeof(struct dialog_item));
+ 	if (!(d = mem_alloc(sizeof(struct dialog) + 14 * sizeof(struct dialog_item)))) return;
+ 	memset(d, 0, sizeof(struct dialog) + 14 * sizeof(struct dialog_item));

It's not actually "cutting" anything, that was just some poor slang for "works".

------- Comment #11 From Seemant Kulleen (RETIRED) 2005-03-23 13:51:46 0000 -------
Marcelo, let's talk on irc, if you would.  I'm online usually from 7am to 5pm
in the EST time zone.  In the meantime, I've bzipped the patch and sent it to
our mirrors.  Once it reaches those, I'll check in the version bump. Thanks for
your patience everyone

------- Comment #12 From Seemant Kulleen (RETIRED) 2005-03-25 05:55:51 0000 -------
pre17 in portage with vanquirius as its new maintainer.  Ingo, please file
future links bugs to him now. Thanks for all your help over the time I've been
looking after this package.

------- Comment #13 From Ingo -LaSombra- Hoffmann 2005-03-27 10:24:58 0000 -------
Hey Seemant,

We're here to help you, developers, to make Gentoo as great as it should be.
:)

------- Comment #14 From Ingo -LaSombra- Hoffmann 2005-03-27 10:25:26 0000 -------
Hey Seemant,

We're here to help you, developers, to make Gentoo as great as it should be.
:)