Opened on 03/20/2015 at 02:33:00 AM

Closed on 04/23/2015 at 01:10:07 AM

#2183 closed defect (fixed)

Create gae2django and rietveld clone

Reported by: matze Assignee:
Priority: P3 Milestone:
Module: Infrastructure Keywords:
Cc: palant, fhd, fred Blocked By:
Blocking: #2182 Platform: Unknown
Ready: yes Confidential: no
Tester: Verified working: no
Review URL(s):

http://codereview.adblockplus.org/6473416005648384/

Description (last modified by trev)

In order to ensure the availability of the Rietveld source for our infrastructure, we have to clone the upstream repositories at https://github.com/andialbrecht/django-gae2django and https://github.com/rietveld-codereview/rietveld into https://hg.adblockplus.org/gae2django and https://hg.adblockplus.org/rietveld respectively - ideally prior to migrating to our own instance.

Attachments (0)

Change History (7)

comment:1 Changed on 03/20/2015 at 02:52:38 AM by matze

Just realized the same should be done for https://code.google.com/p/django-gae2django/ as well. There's no need for an explicit ticket, this can be done in the context of this very issue here.

comment:2 Changed on 04/15/2015 at 11:36:59 AM by matze

  • Description modified (diff)
  • Summary changed from Create Rietveld Clone to Create gae2django and rietveld clone
  • Verified working unset

comment:3 Changed on 04/15/2015 at 12:34:04 PM by trev

  • Description modified (diff)

comment:4 Changed on 04/15/2015 at 01:11:56 PM by matze

Repositories have been created, initial commits have been made:

comment:5 Changed on 04/15/2015 at 08:34:53 PM by matze

For documentation purpose:

It seems like gae2django has been discontinued in May 2012. Thus, recent versions of rietveld cannot be used. In particular, I have found two major issues that reoccur at installation time:

  • The patch-files in gae2django/examples/rietveld/patches fail to apply. Especially patches for rietveld template-files are a problem, because here the most changes have been applied in the last years.
  • Some symbols provided by the Google AppEngine and used by rietveld are not available, e.g. {google,gaeapi}.appengine.db.StringListProperty. In fact there are even rietveld versions prior to 2012 that use this class, so this has been a missing feature in gae2django before.

I've nailed it down to 214 versions of rietveld where none of the above issues occur (according to shell-foo, I have not tested them all). The most recent of those is from November 2012. There've been a few minor issues, like missing variables in the settings.py file, but those could easily get solved (see the codereview later).

comment:6 Changed on 04/15/2015 at 08:40:50 PM by matze

  • Cc palant fhd fred added
  • Review URL(s) modified (diff)
  • Status changed from new to reviewing

comment:7 Changed on 04/23/2015 at 01:10:07 AM by matze

  • Resolution set to fixed
  • Status changed from reviewing to closed

Add Comment

Modify Ticket

Change Properties
Action
as closed .
The resolution will be deleted. Next status will be 'reopened'.
to The owner will be changed from (none).
 
Note: See TracTickets for help on using tickets.