bugThe nmh Mail Handling System - Bugs: bug #18652, nmh-1.2: Add configure flag to...

 
 

bug #18652: nmh-1.2: Add configure flag to disable context/seq file locking

Submitted by:  Craig Leres <leres>
Submitted on:  Sat 30 Dec 2006 11:01:12 PM UTC  
 
Category: EnhancementSeverity: 3 - Normal
Priority: 5 - NormalStatus: Wont Fix
Assigned to: NoneOpen/Closed: Closed

Add a New Comment(Rich Markup)
   

You are not logged in

Please log in, so followups can be emailed to you.

 

Fri 29 Jun 2012 03:09:53 AM UTC, comment #3:

The locking style is now configurable:

--with-locking=[dot|fcntl|flock|lockf]

and at least one of those should be compatible with NFS.

David Levine <levine>
Project Member
Tue 10 Apr 2007 07:40:15 AM UTC, comment #2:

I want to revise all the nmh locking and hopefully a patch like this won't be necessary.

The context and sequences files need to be locked even when being read because they may contain invalid data during modification by another program.

The alternative is to move to a paradigm of replacing the files rather than modifying them, but I don't think that's likely because the MH utilities might not be the only programs working with these files.

Joel Reicher <jjr>
Project Member
Sat 30 Dec 2006 11:26:02 PM UTC, comment #1:

Sorry, I should have changed acconfig.h not config.h.in so I'm attaching a patch for that file.

(file #11645)

Craig Leres <leres>
Sat 30 Dec 2006 11:01:12 PM UTC, original submission:

Between 1.0.4 and 1.2, code was added to do file locking of the context and seq files. This causes problems in some NFS environments; here's a patch to add a configure flag that disables this locking.

As a side note, is it really necessary and/or desirable to lock the context/seq file when just reading it? If not, then the places in context_read.c and seq_read.c could be changed to just do fopen/fclose.

Craig Leres <leres>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach File(s):
   
   
Comment:
   

Attached Files
file #11645:  acconfig.h.patch added by leres (497B - application/octet-stream)
file #11640:  context.patch added by leres (5KiB - application/octet-stream)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by levine (Posted a comment)
  • -unavailable- added by pm215 (Updated the item)
  • -unavailable- added by jjr (Posted a comment)
  • -unavailable- added by leres (Submitted the item)
  •  

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    Follow 6 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Fri 29 Jun 2012 03:09:53 AM UTClevineStatusNone=>Wont Fix
      Open/ClosedOpen=>Closed
    Mon 29 Dec 2008 10:44:22 PM UTCpm215CategoryNone=>Enhancement
    Sat 30 Dec 2006 11:26:02 PM UTCleresAttached File-=>Added acconfig.h.patch, #11645
    Sat 30 Dec 2006 11:01:12 PM UTCleresAttached File-=>Added context.patch, #11640
      Carbon-Copy-=>Added -unavailable-

    Back to the top


    Powered by Savane 3.1-cleanup1