[orca-devel] Flag Day: The refactoring has begun



Hey gang.

The unstable branch may start to live up to its name. We hope not, but we can use your help testing. Important: Before you run off to pull the latest from master, please read this message:

1. You will want to be sure to uninstall -- or otherwise remove -- your
   currently-installed Orca *if* you plan to install the current master
   into the same location.

2. Default.py has been moved to within the scripts dir. This is one of
   the reasons item number one matters. If you have local scripts
   and/or an orca-customizations.py and/or any app settings files which
   import the default script, these will need to updated.

3. presentation_manager.py and focus_tracking_presenter.py are no more.
   The have been removed. Long ago, and before my time working on Orca,
   I believe the plan was to have multiple types of "presentation
   managers," one of which was the focus tracking presenter. Still
   hasn't happened, nor can I think of what new presentation manager
   we'd create. <shrugs>

4. There is a new event_manager.py and a new script_manager.py. They do
   as advertised. And they are where much of what used to be in the
   focus tracking presenter wound up. But mind you, I didn't copy and
   paste. I read each line of code, researched the history behind some
   of it, thought carefully about it all. And then tore much of it
   apart and reworked it, removing what was obsolete or duplicated.
   Things seem a tad snappier as a result, too.

5. Script mapping is done differently; it's a function of the script
   manager and no longer in settings.py.

6. Our first attempt at supporting multiple versions of a script will
   be with Yelp. Yelp 2 and Yelp 3. The former is gecko-based; the
   latter is WebKitGtk based. I'm guessing no one has the latter yet.
   But we need to be ready for those of you who wind up with it.
   The changes made should hopefully happily detect and work with
   whichever version you are using. By the way, this necessitated
   moving the current yelp script out from its original location.

I think those are all the big changes. There were some smaller ones along the way too. But as you're probably concluding, "There is no way on earth Joanie could have done all that without messing *something* up." <grins> I've tested, I've regression tested, Ale has reviewed, but, yeah. I'm sure I missed something. So for those of you who have non-production environments in which you can beat on these changes and let me know what you discover, it would be double-plus awesome.

After I send this, I will push the changes.

Thanks guys! Take care.
--joanie


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]