all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* [Patch] url-cookie-write-file saves versioned backups
@ 2006-08-27 23:53 Diane Murray
  2006-08-28 22:10 ` Richard Stallman
  2006-08-29 14:01 ` Stefan Monnier
  0 siblings, 2 replies; 7+ messages in thread
From: Diane Murray @ 2006-08-27 23:53 UTC (permalink / raw)


Versioned backups of the cookies file are being saved (contrary to the
intent of the last changes made to url-cookie.el in revision 1.18),
which causes `backup-buffer' to ask for confirmation to delete those
excess backup files.  Additionally, the error "Local variables list is
not properly terminated" occurs when opening the cookies file.  The
following patch corrects this.

I submitted earlier patches using the email <dsm@muenster.de>, if for
some reason that needs to be known.


	* url-cookie.el (url-cookie-write-file): Insert a newline
	character before the Local Variables line.  Set `version-control'
	to never.


*** url-cookie.el	13 Jul 2006 03:53:15 +0200	1.18
--- url-cookie.el	27 Aug 2006 15:25:21 +0200	
***************
*** 168,178 ****
        (insert ")\n(setq url-cookie-secure-storage\n '")
        (pp url-cookie-secure-storage (current-buffer))
        (insert ")\n")
!       (insert "\f;; Local Variables:\n"
                ";; version-control: never\n"
                ";; no-byte-compile: t\n"
                ";; End:\n")
!       (set (make-local-variable 'version-control) t)
        (write-file fname)
        (setq url-cookies-changed-since-last-save nil)
        (kill-buffer (current-buffer))))))
--- 168,178 ----
        (insert ")\n(setq url-cookie-secure-storage\n '")
        (pp url-cookie-secure-storage (current-buffer))
        (insert ")\n")
!       (insert "\f\n;; Local Variables:\n"
                ";; version-control: never\n"
                ";; no-byte-compile: t\n"
                ";; End:\n")
!       (set (make-local-variable 'version-control) 'never)
        (write-file fname)
        (setq url-cookies-changed-since-last-save nil)
        (kill-buffer (current-buffer))))))

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [Patch] url-cookie-write-file saves versioned backups
  2006-08-27 23:53 [Patch] url-cookie-write-file saves versioned backups Diane Murray
@ 2006-08-28 22:10 ` Richard Stallman
  2006-08-29  0:13   ` Diane Murray
  2006-08-29  5:01   ` Eric Hanchrow
  2006-08-29 14:01 ` Stefan Monnier
  1 sibling, 2 replies; 7+ messages in thread
From: Richard Stallman @ 2006-08-28 22:10 UTC (permalink / raw)
  Cc: emacs-devel

    Versioned backups of the cookies file are being saved (contrary to the
    intent of the last changes made to url-cookie.el in revision 1.18),
    which causes `backup-buffer' to ask for confirmation to delete those
    excess backup files.

Is that necessarily wrong?

      Additionally, the error "Local variables list is
    not properly terminated" occurs when opening the cookies file.  The
    following patch corrects this.

We should certainly fix that one.
Would someone install that part?

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [Patch] url-cookie-write-file saves versioned backups
  2006-08-28 22:10 ` Richard Stallman
@ 2006-08-29  0:13   ` Diane Murray
  2006-08-29 17:18     ` Richard Stallman
  2006-08-29  5:01   ` Eric Hanchrow
  1 sibling, 1 reply; 7+ messages in thread
From: Diane Murray @ 2006-08-29  0:13 UTC (permalink / raw)
  Cc: emacs-devel

Richard Stallman <rms@gnu.org> wrote:

>     Versioned backups of the cookies file are being saved (contrary
>     to the intent of the last changes made to url-cookie.el in
>     revision 1.18), which causes `backup-buffer' to ask for
>     confirmation to delete those excess backup files.

> Is that necessarily wrong?

I don't know why the original changes were made to disable version
control for the cookies file, but I assume that if it is to be done,
`version-control' as `set' locally in the buffer is supposed to have
the same value as the one inserted into the Local Variables section of
the file.  Right now the former is set to t and the latter is set to
never.

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [Patch] url-cookie-write-file saves versioned backups
  2006-08-28 22:10 ` Richard Stallman
  2006-08-29  0:13   ` Diane Murray
@ 2006-08-29  5:01   ` Eric Hanchrow
  2006-08-29 17:18     ` Richard Stallman
  1 sibling, 1 reply; 7+ messages in thread
From: Eric Hanchrow @ 2006-08-29  5:01 UTC (permalink / raw)


>>>>> "Richard" == Richard Stallman <rms@gnu.org> writes:

    Richard>     Versioned backups of the cookies file are being saved
    Richard>     (contrary to the intent of the last changes made to
    Richard>     url-cookie.el in revision 1.18), which causes
    Richard>     `backup-buffer' to ask for confirmation to delete
    Richard>     those excess backup files.

    Richard> Is that necessarily wrong?

I think so, because the confirmation prompts seem to appear
asynchronously -- I'll be doing something that has nothing to do with
w3m (like using ERC), and suddenly I'll be asked to delete excess
backups.  And this happens fairly frequently -- perhaps every half
hour.

-- 
But users will not now with glad cries glom on to a language that
gives them no more than what Scheme or Pascal gave them.
        -- Guy Steele, http://www.sun.com/research/jtech/pubs/98-oopsla-growing.ps

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [Patch] url-cookie-write-file saves versioned backups
  2006-08-27 23:53 [Patch] url-cookie-write-file saves versioned backups Diane Murray
  2006-08-28 22:10 ` Richard Stallman
@ 2006-08-29 14:01 ` Stefan Monnier
  1 sibling, 0 replies; 7+ messages in thread
From: Stefan Monnier @ 2006-08-29 14:01 UTC (permalink / raw)
  Cc: emacs-devel

> Versioned backups of the cookies file are being saved (contrary to the
> intent of the last changes made to url-cookie.el in revision 1.18),
> which causes `backup-buffer' to ask for confirmation to delete those
> excess backup files.  Additionally, the error "Local variables list is
> not properly terminated" occurs when opening the cookies file.  The
> following patch corrects this.

I'm sorry I messed up the earlier fix, and thank you for setting
me straight.  I've just installed your patch,


        Stefan

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [Patch] url-cookie-write-file saves versioned backups
  2006-08-29  0:13   ` Diane Murray
@ 2006-08-29 17:18     ` Richard Stallman
  0 siblings, 0 replies; 7+ messages in thread
From: Richard Stallman @ 2006-08-29 17:18 UTC (permalink / raw)
  Cc: emacs-devel

    I don't know why the original changes were made to disable version
    control for the cookies file, but I assume that if it is to be done,
    `version-control' as `set' locally in the buffer is supposed to have
    the same value as the one inserted into the Local Variables section of
    the file.  Right now the former is set to t and the latter is set to
    never.

It does seem like a bug to specify two different values for the same
variable in two different ways.  So we should change something.
The question that remains is what value to use.

Who put in the two specifications for these two values,
and what were their motives, and who else saw those reasons and
agreed with them or opposed them.

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [Patch] url-cookie-write-file saves versioned backups
  2006-08-29  5:01   ` Eric Hanchrow
@ 2006-08-29 17:18     ` Richard Stallman
  0 siblings, 0 replies; 7+ messages in thread
From: Richard Stallman @ 2006-08-29 17:18 UTC (permalink / raw)
  Cc: emacs-devel

    I think so, because the confirmation prompts seem to appear
    asynchronously -- I'll be doing something that has nothing to do with
    w3m (like using ERC), and suddenly I'll be asked to delete excess
    backups.  And this happens fairly frequently -- perhaps every half
    hour.

I think that is a good argument, but before deciding, let's give
people time to make other arguments.

^ permalink raw reply	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2006-08-29 17:18 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2006-08-27 23:53 [Patch] url-cookie-write-file saves versioned backups Diane Murray
2006-08-28 22:10 ` Richard Stallman
2006-08-29  0:13   ` Diane Murray
2006-08-29 17:18     ` Richard Stallman
2006-08-29  5:01   ` Eric Hanchrow
2006-08-29 17:18     ` Richard Stallman
2006-08-29 14:01 ` Stefan Monnier

Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/emacs.git
	https://git.savannah.gnu.org/cgit/emacs/org-mode.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.