On 12-10-12 05:50 PM, John King wrote: > Several issues arise with this fix that I, as a new developer, would > like some feedback on: > 1) Can this fix go into the 1.4 release? UC Berkeley would like to see > it in the release as YouTube is one of our main distribution points.
My understanding of this is that it worked previously, but does not currently work in 1.4? This would make it a bug, so I don't see any reason why it couldn't be merged in. > 2) Related to #1, regression testing - I am not sure that I know the app > well enough to do that comprehensively. I do know it fixes the YouTube > problem and seems low risk. One of the biggest issues we face is testing stuff like this. The policy so far as been that it can go in as long as it doesn't break any tests. I don't think, however, that we have any tests covering Youtube distribution... > 3) Handling and use of the combined Bundle itself. Here is a pom.xml > entry for it: > <dependency> > <groupId>org.sakaiproject.nakamura</groupId> > <artifactId>javax.activation-mail</artifactId> > <version>1.1.1.1.4.2-1.3.0</version> > </dependency> > > The options I see for use of it are: > > * Install the bundle in the Matterhorn maven repository - my > preferred option > * Add the Sakai repository to our list of maven repositories and > pull it from there > * Build a new bundle from scratch that will be a matterhorn bundle > > I did chat with Carl Hall, the developer who built it and is a > former colleague of mine from the Sakai Nakamura project. He said > that licensing will permit option 1 and he agreed with me that > having control over it, e.g. installing it in the Mattehorn repo, is > probably the best option. Do you have more information about the licensing? We have a few policy documents in the wiki so this should be easy enough to sort out. > 4) What to do about the matterhorn-composer use of the service-mix mail > bundle? I'll bring this up tomorrow. I'm not sure what composer is using that for, but it might just be a dead dependency. G > There may likely be other issues I am overlooking so please address any > you see. > > Thanks in advance for feedback. > > > > _______________________________________________ > Matterhorn mailing list > [email protected] > http://lists.opencastproject.org/mailman/listinfo/matterhorn > > > To unsubscribe please email > [email protected] > _______________________________________________ >
signature.asc
Description: OpenPGP digital signature
_______________________________________________ Matterhorn mailing list [email protected] http://lists.opencastproject.org/mailman/listinfo/matterhorn To unsubscribe please email [email protected] _______________________________________________
