From 74c2b8c0126a054efd696f152fd46352015c6973 Mon Sep 17 00:00:00 2001
From: Richard Darst <rkd@zgib.net>
Date: Sun, 23 Jan 2022 02:54:40 +0200
Subject: [PATCH] Don't set end=None for recurring events

- This seems to make the events be zero-length, which is probably not
  intended.  At least with duration=, recurring events work as
  expected with this change.

- Review: someone who knows more about ics and why this was made this
  way should take a look.
---
 yaml2ics.py | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/yaml2ics.py b/yaml2ics.py
index 7796c60..e57d21d 100644
--- a/yaml2ics.py
+++ b/yaml2ics.py
@@ -55,7 +55,8 @@ def event_from_yaml(event_yaml: dict) -> ics.Event:
             print('Error: must specify end date for repeating events', file=sys.stderr)
             sys.exit(-1)
 
-        event.end = d.get('end', None)
+        # This causes zero-length events, I guess overriding whatever duration might have been specified.
+        #event.end = d.get('end', None)
 
         rrule = dateutil.rrule.rrule(
             freq=interval_type[interval_measure],