Skip to content
This repository has been archived by the owner. It is now read-only.

calendarserver_export uses CalDAVFile #186

Closed
macosforgebot opened this issue Feb 5, 2011 · 7 comments
Closed

calendarserver_export uses CalDAVFile #186

macosforgebot opened this issue Feb 5, 2011 · 7 comments

Comments

@macosforgebot
Copy link

@macosforgebot macosforgebot commented Feb 5, 2011

mail@… originally submitted this as ticket:406


Current trunk @r6886 uses in calendarserver_export the CalDAVFile from static.py.

This has been reomoved 6 months ago.

http://trac.calendarserver.org/browser/CalendarServer/trunk/twistedcaldav/static.py?rev=5966

@macosforgebot
Copy link
Author

@macosforgebot macosforgebot commented Feb 5, 2011

mail@… originally submitted this as comment:1:⁠ticket:406


The error message one gets for google to index:

Traceback (most recent call last):
  File "/usr/bin/calendarserver_export", line 43, in <module>
    from calendarserver.tools.export import main
  File "/usr/lib/python2.7/site-packages/calendarserver/tools/export.py", line 44, in <module>
    from twistedcaldav.static import CalDAVFile
ImportError: No module named static
@macosforgebot
Copy link
Author

@macosforgebot macosforgebot commented Feb 5, 2011

mail@… originally submitted this as comment:2:⁠ticket:406


--- ./calendarserver/tools/export.py    2011-02-04 21:20:12.824330578 +0100
+++ /usr/lib/python2.7/site-packages/calendarserver/tools/export.py     2011-02-05 02:38:16.531756689 +0100
@@ -41,7 +41,7 @@
 from twistedcaldav.ical import iCalendarProductID
 from twistedcaldav.resource import isCalendarCollectionResource,\
     CalendarHomeResource
-from twistedcaldav.static import CalDAVFile
+from twistedcaldav.resource import CalDAVResource
 from twistedcaldav.directory.directory import DirectoryService

 from calendarserver.tools.util import UsageError
@macosforgebot
Copy link
Author

@macosforgebot macosforgebot commented Feb 5, 2011

mail@… originally submitted this as comment:3:⁠ticket:406


My fix does not really solve the problem which requires way more work:

<fmms> glyph, just patched that have a look at the ticket
<glyph> fmms: it's not quite that simple
<fmms> at least then i can read the help ;)
<glyph> fmms: the data-store API is completely different now, CalDAVResource isn't backed by files on disk.  Even if you use the (unsupported) filesystem backend, it has a different API.
<glyph> yeah that tool just hasn't been updated yet
<glyph> it's not a matter of a single import, it needs to be rewritten :)
<fmms> glyph, i am just using the filesystem backend, why is it not supported?
<fmms> what does that mean, will you remove it any day, or will bugs be fixed slower?
<glyph> fmms: It means that you should use the database backend because that's the one that's supposed to work :).
@macosforgebot
Copy link
Author

@macosforgebot macosforgebot commented Apr 5, 2011

@wsanchez originally submitted this as comment:4:⁠ticket:406

  • Owner changed from @wsanchez to @m0rgen
  • Priority changed from 5: Not set to 1: Blocker
  • Milestone set to CalendarServer-3.0
@macosforgebot
Copy link
Author

@macosforgebot macosforgebot commented May 16, 2011

mail@… originally submitted this as comment:5:⁠ticket:406


I think this issue has been fixed at least partly by commit r7486 ??

One question why does this use /etc/cardavd/caldavd.plist instead of /etc/caldavd/caldavd.plist?

@macosforgebot
Copy link
Author

@macosforgebot macosforgebot commented May 16, 2011

@glyph originally submitted this as comment:6:⁠ticket:406


Replying to mail@…:

I think this issue has been fixed at least partly by commit r7486 ??

One question why does this use /etc/cardavd/caldavd.plist instead of /etc/caldavd/caldavd.plist?

Good catch! That was autocomplete gone wrong.

@macosforgebot
Copy link
Author

@macosforgebot macosforgebot commented May 16, 2011

@glyph originally submitted this as comment:7:⁠ticket:406

  • Status changed from new to closed
  • Resolution changed from to Software changed

The wrong-default-config-file issue fixed in http://trac.calendarserver.org/changeset/7489.

Thanks for reporting the bug!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
3 participants
You can’t perform that action at this time.