Nick Dokos <nicholas.dokos@hp.com> writes:

> Samuel Loury <konubinix@gmail.com> wrote:
>
>> ╭────
>> │ (defun my/org-last-week ()
>> │   (- (org-today) 7)
>> │   )
>> │ 
>> │ 
>> │ (add-to-list 'org-agenda-custom-commands
>> │ 			 '("w" "Weekly Logs"
>> │ 			   (
>> │ 				(agenda nil
>> │ 						(
>> │ 						 (org-agenda-overriding-header
>> │ 						  "Review for last week")
>> │ 						 (org-agenda-span 8)
>> │ 						 )
>> │ 						)
>> │ 				)
>> │ 			   (
>> │ 				(org-agenda-start-day 'my/org-last-week)
>
> I think that's wrong: there is no support afaik for this variable to have
> a function value.

Hum. You're right. I forgot I also patched the org-agenda.el code for
this to work.

> Should that  be
>
>  				(org-agenda-start-day (my/org-last-week))

That works well indeed. And does not need any patch.
Thanks for the correction.

-- 
Konubinix
GPG Key    : 7439106A
Fingerprint: 5993 BE7A DA65 E2D9 06CE  5C36 75D2 3CED 7439 106A