Opened 8 years ago

Last modified 4 years ago

#22228 reopened defect

nvi says "page sizes must be a power-of-2"

Reported by: helbig.wolfgang@… Owned by: tobypeterson
Priority: Low Milestone:
Component: ports Version: 1.8.1
Keywords: Cc: qbarnes@…, suominen (Kimmo Suominen)
Port: nvi

Description

This occurs when editing a big file. I do not know, whether its harmful. But makes me feel uncomfortable somewhat.

Attachments (1)

nvi.patch (1.6 KB) - added by qbarnes@… 7 years ago.
Patch to address "page size must be power-of-2" message.

Download all attachments as: .zip

Change History (8)

comment:1 Changed 8 years ago by jmroot (Joshua Root)

  • Keywords nvi pages size power-of-2 removed
  • Owner changed from macports-tickets@… to toby@…
  • Port nvi added

comment:2 Changed 8 years ago by tobypeterson

  • Resolution set to wontfix
  • Status changed from new to closed

we don't have a "third party to resolve" resolution, but seriously - report this upstream

comment:3 Changed 8 years ago by helbig.wolfgang@…

Hmm, since nvi works flawlessly on NetBSD, this looked like I porting issue to me.

Changed 7 years ago by qbarnes@…

Patch to address "page size must be power-of-2" message.

comment:4 Changed 7 years ago by qbarnes@…

  • Resolution wontfix deleted
  • Status changed from closed to reopened

The message comes from db4. Nvi is requesting a non-power of 2 page size with its call to db->set_pagesize(). The message comes out on OSX as well as Linux. I don't know why NetBSD doesn't get the same message. I haven't looked.

I developed a fix that works to address this issue on both Linux and OSX. I uploaded a patch for Macports nvi that I installed and tested. Seems to work fine. I wish I understood the code better, but the fix does solve the problem and I think doesn't introduce any other side effects or additional issues. Let me know if there's any problems applying or using it.

comment:5 Changed 7 years ago by qbarnes@…

  • Cc qbarnes@… added

Cc Me!

comment:6 Changed 7 years ago by tobypeterson

  • Priority changed from Normal to Low

comment:7 Changed 4 years ago by suominen (Kimmo Suominen)

  • Cc kimmo@… added

Cc Me!

Note: See TracTickets for help on using tickets.