Move console scripts from osi to karl

Bug #391247 reported by Chris Rossi
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
KARL3
Fix Released
Medium
Shane Hathaway

Bug Description

Most, if not all, console scripts currently in osi.scripts can probably be moved to karl.

Revision history for this message
Chris Rossi (chris-archimedeanco) wrote :

This is part of a series of tickets that will involve looking at various "customizations" that have been performed in the osi package for Karl3. Many of these are things that are probably more generally useful than just for osi and the hope is that many of these things can be moved into the karl or karl.content packages as appropriate. The idea here is to pare down to a bare minimum the amount of code that actually lives in the osi package so we can have a better idea of what customizations will be necessary for accomodating other pilots.

For each of these items there will be a judgment call in terms of assigning to one of these categories:

o It is truly a customization and should only ever live in the osi package.

o It is generally useful, should be part of core Karl and can be moved to either karl or karl.content.

o It is generally useful but has some small osi tweak and so should be refactored such that the really useful bits live in karl and osi has only a small tweak, either through configuration or subclassing or some other means.

The hope is most of these will be in the second category.

Changed in karl3:
assignee: nobody → Chris Rossi (chris-archimedeanco)
importance: Undecided → Medium
Changed in karl3:
milestone: none → m20
Revision history for this message
Chris Rossi (chris-archimedeanco) wrote :

Hi Shane, do you want to take this one? It's not super important to me whether the entry points continue to live in osi or if they also move to Karl. Use your judgment on that one.

Changed in karl3:
assignee: Chris Rossi (chris-archimedeanco) → Shane Hathaway (shane-hathawaymix)
Revision history for this message
Chris Rossi (chris-archimedeanco) wrote :

Another week, another milestone.

Changed in karl3:
milestone: m20 → m21
Revision history for this message
Shane Hathaway (shane-hathawaymix) wrote :

I moved as many as I could without any redesign work. Scripts left over:

- The scripts involved in the Karl 2 -> Karl 3 migration, since OSI is the only org that ever had Karl 2 (as far as I know). These include aclimport, bulkgenerator, bulkloader, fix_userids, importer, local_roles_report, and scrub_content. Perhaps we should just delete these now that the Karl 2 migration is over.

- evolve and startover, which are generic ideas but they refer to OSI-specific code. Making them generic would require some redesign. They are small and not used often, so I think it's better to leave them alone for now.

Changed in karl3:
status: New → Fix Committed
Revision history for this message
Chris Rossi (chris-archimedeanco) wrote : Re: [Bug 391247] Re: Move console scripts from osi to karl

On Tue, Jun 30, 2009 at 1:37 AM, Shane Hathaway <email address hidden>wrote:

> I moved as many as I could without any redesign work. Scripts left over:
>
> - The scripts involved in the Karl 2 -> Karl 3 migration, since OSI is
> the only org that ever had Karl 2 (as far as I know). These include
> aclimport, bulkgenerator, bulkloader, fix_userids, importer,
> local_roles_report, and scrub_content. Perhaps we should just delete
> these now that the Karl 2 migration is over.
>

Actually, Six Feet Up will be migrating Oxfam and Eurasia from Karl2 to
Karl3 in July, using this code.

>
> - evolve and startover, which are generic ideas but they refer to OSI-
> specific code. Making them generic would require some redesign. They are
> small and not used often, so I think it's better to leave them alone for
> now.
>
>
Right. We'll need to duplicate these for other pilots with pilot specific
code.

Thanks,
Chris

Revision history for this message
Shane Hathaway (shane-hathawaymix) wrote :

After discussion, we decided scripts that require a config file belong in the osi package after all. So we moved the body of the larger scripts to the karl package and added tests.

Changed in karl3:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.