Opened 14 years ago

Closed 4 years ago

#22228 closed defect (fixed)

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 (Quentin Barnes), 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 (Quentin Barnes) 14 years ago.
Patch to address "page size must be power-of-2" message.

Download all attachments as: .zip

Change History (10)

comment:1 Changed 14 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 14 years ago by tobypeterson

Resolution: wontfix
Status: newclosed

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

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

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

Changed 14 years ago by qbarnes (Quentin Barnes)

Attachment: nvi.patch added

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

comment:4 Changed 14 years ago by qbarnes (Quentin Barnes)

Resolution: wontfix
Status: closedreopened

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 14 years ago by qbarnes (Quentin Barnes)

Cc: qbarnes@… added

Cc Me!

comment:6 Changed 13 years ago by tobypeterson

Priority: NormalLow

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

Cc: kimmo@… added

Cc Me!

comment:8 Changed 4 years ago by tobypeterson

Still appears to be reproducible; not sure the proposed patch is the best approach.

Last edited 4 years ago by tobypeterson (previous) (diff)

comment:9 Changed 4 years ago by tobypeterson

Resolution: fixed
Status: reopenedclosed

In 572347846879b23679491e9e36dd15da2f6ca27b/macports-ports (master):

nvi: fix db4 pagesize warning
fixes #22228

Note: See TracTickets for help on using tickets.