[jira] [Commented] (SLING-2002) Migrate the Sling website to the ASF CMS

2012-04-03 Thread Bertrand Delacretaz (Commented) (JIRA)

[ 
https://issues.apache.org/jira/browse/SLING-2002?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13245439#comment-13245439
 ] 

Bertrand Delacretaz commented on SLING-2002:


And note also the other adjustments done in the current confluence export 
script [1]:

# copy downloads page and replace with patched
cp ${DLPAGE} ${DLTMP}
cat ${DLTMP} | sed  
's/http:\/\/cwiki.apache.org\/confluence\/display\/SLINGxSITE\/%5Bpreferred%5D/[preferred]/g'
 > ${DLPAGE}

# copy apache-sling.html to index.html
cp ${SITE}/apache-sling.html ${SITE}/index.html

[1] http://sling.apache.org/site/documentation.data/sling.sh

> Migrate the Sling website to the ASF CMS
> 
>
> Key: SLING-2002
> URL: https://issues.apache.org/jira/browse/SLING-2002
> Project: Sling
>  Issue Type: Task
>  Components: Site
>Reporter: Bertrand Delacretaz
>Assignee: Bertrand Delacretaz
>
> As discussed at http://markmail.org/thread/vbdqqnbaghpcxwp5
> I'll try the tools mentioned at 
> http://wiki.apache.org/general/ApacheCms2010#Migrating_from_Confluence

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira




[jira] [Commented] (SLING-2002) Migrate the Sling website to the ASF CMS

2012-04-03 Thread Bertrand Delacretaz (Commented) (JIRA)

[ 
https://issues.apache.org/jira/browse/SLING-2002?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13245434#comment-13245434
 ] 

Bertrand Delacretaz commented on SLING-2002:


About downloads.cgi, as described at SLING-1425 it seems like we just need to 
add an executable downloads.cgi script alongside downloads.html:

!/bin/sh
# Wrapper script around mirrors.cgi script
# (we must change to that directory in order for python to pick up the
#  python includes correctly)
cd /www/www.apache.org/dyn/mirrors
/www/www.apache.org/dyn/mirrors/mirrors.cgi $*

And once we switch we must also make sure the static website parts currently 
found on people.a.o:/www/sling.apache.org are also copied to the "new" site.


> Migrate the Sling website to the ASF CMS
> 
>
> Key: SLING-2002
> URL: https://issues.apache.org/jira/browse/SLING-2002
> Project: Sling
>  Issue Type: Task
>  Components: Site
>Reporter: Bertrand Delacretaz
>Assignee: Bertrand Delacretaz
>
> As discussed at http://markmail.org/thread/vbdqqnbaghpcxwp5
> I'll try the tools mentioned at 
> http://wiki.apache.org/general/ApacheCms2010#Migrating_from_Confluence

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira




[jira] [Commented] (SLING-2002) Migrate the Sling website to the ASF CMS

2012-04-03 Thread Bertrand Delacretaz (Commented) (JIRA)

[ 
https://issues.apache.org/jira/browse/SLING-2002?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13245391#comment-13245391
 ] 

Bertrand Delacretaz commented on SLING-2002:


In the meantime, Dan Kulp and the Camel/CXF team has come up with a 
Maven-driven rendering of Confluence websites. With this we could continue 
editing our content in Confluence, and have it published much more quickly to 
sling.apache.org.

I have started experimenting with this at 
https://svn.apache.org/repos/asf/sling/whiteboard/cwiki-to-svn, in revision 
1308967 the export works (with no real CSS yet), and updates are correctly 
detected when running the export again 1-2 minutes after changing a page in 
Confluence.

One thing to look at is the rendering of the 
http://sling.apache.org/site/downloads.cgi page - I'm not sure at this what's 
needed to make it dynamic on the live website.

See also:
http://www.dankulp.com/blog/2012/03/svnpubsub-for-confluence-sites/
http://www.apache.org/dev/project-site.html


> Migrate the Sling website to the ASF CMS
> 
>
> Key: SLING-2002
> URL: https://issues.apache.org/jira/browse/SLING-2002
> Project: Sling
>  Issue Type: Task
>  Components: Site
>Reporter: Bertrand Delacretaz
>Assignee: Bertrand Delacretaz
>
> As discussed at http://markmail.org/thread/vbdqqnbaghpcxwp5
> I'll try the tools mentioned at 
> http://wiki.apache.org/general/ApacheCms2010#Migrating_from_Confluence

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira




[jira] [Commented] (SLING-1725) Register internal post operations as services for consumption by servlets other than the SlingPostServlet

2012-04-03 Thread Bertrand Delacretaz (Commented) (JIRA)

[ 
https://issues.apache.org/jira/browse/SLING-1725?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13245294#comment-13245294
 ] 

Bertrand Delacretaz commented on SLING-1725:


I have also updated the "Extending the SlingPostServlet" docs at 
http://sling.apache.org/site/manipulating-content-the-slingpostservlet-servletspost.html
 to reflect this change, should be online soon

> Register internal post operations as services for consumption by servlets 
> other than the SlingPostServlet
> -
>
> Key: SLING-1725
> URL: https://issues.apache.org/jira/browse/SLING-1725
> Project: Sling
>  Issue Type: Improvement
>  Components: Servlets
>Affects Versions: Servlets Post 2.0.4
>Reporter: Felix Meschberger
>Assignee: Bertrand Delacretaz
> Fix For: Servlets Post 2.1.2
>
> Attachments: SLING-1725.patch
>
>
> As discussed in [1] it would be useful to have the internal operations of the 
> Sling POST Servlet available as services for other bundles to reuse.
> [1] http://markmail.org/message/a7vrtyhictf7tv4m

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira




[jira] [Commented] (SLING-1437) GSoC 2010: create more unit and integration tests for Sling

2012-04-01 Thread Bertrand Delacretaz (Commented) (JIRA)

[ 
https://issues.apache.org/jira/browse/SLING-1437?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13243695#comment-13243695
 ] 

Bertrand Delacretaz commented on SLING-1437:


If a student has specific skills/interests that match this project we might be 
able to make it happen in 2012 - best is to contact us via the Sling 
developer's mailing list, 
http://sling.apache.org/site/project-information.html#ProjectInformation-lists

> GSoC 2010: create more unit and integration tests for Sling
> ---
>
> Key: SLING-1437
> URL: https://issues.apache.org/jira/browse/SLING-1437
> Project: Sling
>  Issue Type: Task
>  Components: Testing
>Reporter: Bertrand Delacretaz
>  Labels: gsoc, gsoc2010sling, mentor, testing
>
> Sling already has fairly good test coverage, but there's always room for 
> improvement. 
> The goal of this Google Summer of Code project is to create more unit and 
> integration tests for Sling.
> The student will have to get familiar with the Sling codebase, identify areas 
> where tests are missing, write unit and integration tests to improve the test 
> coverage, and submit the result as patches that the Sling committers can 
> verify and hopefully apply.
> Another aspect is measuring and reporting (automatically, ideally as part of 
> our Hudson builds) the actual test coverage - that's easy for unit tests 
> which run as part of the module builds, but seems to be harder for 
> integration tests which run in a separate JVM than the one that executes 
> tests.
> See http://tinyurl.com/asfgsoc for the full list of GSoC 2010 projects at the 
> ASF, and http://community.apache.org/gsoc for general GSoC information.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira




[jira] [Commented] (SLING-2443) Missing WWW-Authenticate header on OPTIONS request with trunk servlets.resolver bundle

2012-03-21 Thread Bertrand Delacretaz (Commented) (JIRA)

[ 
https://issues.apache.org/jira/browse/SLING-2443?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13234392#comment-13234392
 ] 

Bertrand Delacretaz commented on SLING-2443:


I agree about the reset in general, but the following pattern seems reasonable 
to me:

  // Sling servlet is in trouble
  response.setHeader("X-something-specific", "some useful value")
  response.sendError(...)

And if you do this in a Sling servlet now that won't work...maybe we should 
preserve the headers instead of resetting everything?

> Missing WWW-Authenticate header on OPTIONS request with trunk 
> servlets.resolver bundle
> --
>
> Key: SLING-2443
> URL: https://issues.apache.org/jira/browse/SLING-2443
> Project: Sling
>  Issue Type: Bug
>  Components: Servlets
>Reporter: Bertrand Delacretaz
>
> Running the launchpad/builder standalone jar from the trunk correctly returns 
> an WWW-Authenticate header on an OPTIONS request with no credentials:
> $ curl -D - -X OPTIONS http://localhost:8080/
> HTTP/1.1 401 Unauthorized
> WWW-Authenticate: Basic realm="Jackrabbit Webdav Server"
> Content-Type: ...
> But if I replace the org.apache.sling.servlets.resolver 2.1.0 bundle that's 
> in there with the latest snapshot (revision 1302994) that header is missing: 
> it gets removed by the response.reset() call in 
> DefaultErrorHandlerServlet.sendIntro(...), which makes it impossible to 
> connect with WebDAV.
> That response.reset() call was not present in 2.1.0.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira




[jira] [Commented] (SLING-2443) Missing WWW-Authenticate header on OPTIONS request with trunk servlets.resolver bundle

2012-03-20 Thread Bertrand Delacretaz (Commented) (JIRA)

[ 
https://issues.apache.org/jira/browse/SLING-2443?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13233562#comment-13233562
 ] 

Bertrand Delacretaz commented on SLING-2443:


IIUC it's jackrabbit's AbstractWebdavServlet. sendUnauthorized(...) method that 
sends this:

response.setHeader("WWW-Authenticate", getAuthenticateHeaderValue());
if (error == null || error.getErrorCode() != 
HttpServletResponse.SC_UNAUTHORIZED) {
response.sendError(HttpServletResponse.SC_UNAUTHORIZED);
} else {
response.sendError(error.getErrorCode(), error.getStatusPhrase());
}

and that looks a bit tricky to override (just had a quick look)

[1] 
http://svn.apache.org/repos/asf/jackrabbit/trunk/jackrabbit-webdav/src/main/java/org/apache/jackrabbit/webdav/server/AbstractWebdavServlet.java

> Missing WWW-Authenticate header on OPTIONS request with trunk 
> servlets.resolver bundle
> --
>
> Key: SLING-2443
> URL: https://issues.apache.org/jira/browse/SLING-2443
> Project: Sling
>  Issue Type: Bug
>  Components: Servlets
>Reporter: Bertrand Delacretaz
>
> Running the launchpad/builder standalone jar from the trunk correctly returns 
> an WWW-Authenticate header on an OPTIONS request with no credentials:
> $ curl -D - -X OPTIONS http://localhost:8080/
> HTTP/1.1 401 Unauthorized
> WWW-Authenticate: Basic realm="Jackrabbit Webdav Server"
> Content-Type: ...
> But if I replace the org.apache.sling.servlets.resolver 2.1.0 bundle that's 
> in there with the latest snapshot (revision 1302994) that header is missing: 
> it gets removed by the response.reset() call in 
> DefaultErrorHandlerServlet.sendIntro(...), which makes it impossible to 
> connect with WebDAV.
> That response.reset() call was not present in 2.1.0.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira




[jira] [Commented] (SLING-2376) New Startup Features

2012-02-10 Thread Bertrand Delacretaz (Commented) (JIRA)

[ 
https://issues.apache.org/jira/browse/SLING-2376?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13205361#comment-13205361
 ] 

Bertrand Delacretaz commented on SLING-2376:


Just so that this info is complete, Carsten's "more information" email thread 
is at http://markmail.org/message/cikqwmp562vew6pr

> New Startup Features
> 
>
> Key: SLING-2376
> URL: https://issues.apache.org/jira/browse/SLING-2376
> Project: Sling
>  Issue Type: New Feature
>  Components: Launchpad
>Affects Versions: Launchpad Installer 1.0.6, Launchpad Base 2.3.0, 
> Launchpad API 1.0.0
>Reporter: Carsten Ziegeler
>Assignee: Carsten Ziegeler
> Fix For: Launchpad Installer 1.1.0, Launchpad Base 2.4.0, 
> Launchpad API 1.1.0
>
>
> With SLING-2372 we have now a mechanism to decide what kind of startup 
> currently is processed
> We could leverage this to create new functionality:
> Enhance lauchpad api with the startup mode, the possibility to register 
> listener which are informed about the startup mode and progress.
> Another feature would be to not directly go to the beginning start level of 
> the framework, but stop at an intermediate level on installs and updates and 
> from there increase the start level one by one.
> This makes updates and installs much smoother
> For more information I'll write an email to the dev list :)

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira




[jira] [Commented] (SLING-2380) Make RepositoryTestBase's repository usable from JUnit 4.x style tests

2012-01-23 Thread Bertrand Delacretaz (Commented) (JIRA)

[ 
https://issues.apache.org/jira/browse/SLING-2380?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13191263#comment-13191263
 ] 

Bertrand Delacretaz commented on SLING-2380:


Yes, but it doesn't have the on-demand initialization and automatic shutdown 
that RepositoryTestBase provides.

> Make RepositoryTestBase's repository usable from JUnit 4.x style tests
> --
>
> Key: SLING-2380
> URL: https://issues.apache.org/jira/browse/SLING-2380
> Project: Sling
>  Issue Type: Improvement
>  Components: Testing
>Reporter: Bertrand Delacretaz
>Assignee: Bertrand Delacretaz
>Priority: Minor
>
> Tests currently need to inherit from that class to get a TestRepository, 
> which forces them to use the old JUnit3.x style.
> I'll add a static method to that class, that JUnit 4.x tests can use to get a 
> repository.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira




[jira] [Commented] (SLING-2327) Visible TODOs in the background servlets console

2012-01-13 Thread Bertrand Delacretaz (Commented) (JIRA)

[ 
https://issues.apache.org/jira/browse/SLING-2327?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13185648#comment-13185648
 ] 

Bertrand Delacretaz commented on SLING-2327:


> Disabling the plugin by configuration is IMHO not required because the Web 
> Console is targeted at admins only. 

ok, I see your point - I'll leave the setting in for now, I don't think it 
hurts to have it.

> Visible TODOs in the background servlets console
> 
>
> Key: SLING-2327
> URL: https://issues.apache.org/jira/browse/SLING-2327
> Project: Sling
>  Issue Type: Bug
>  Components: Extensions
>Reporter: Bertrand Delacretaz
>Priority: Minor
>
> The console plugin of the contrib/extensions/bgservlets displays some ugly 
> TODOs, those should be removed.
> It might also be good to make this plugin optional, in case users want to 
> supply a custom variant.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira




[jira] [Commented] (SLING-2347) Startup filter: allow requests to be rejected with a 503 during startup

2012-01-04 Thread Bertrand Delacretaz (Commented) (JIRA)

[ 
https://issues.apache.org/jira/browse/SLING-2347?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13179608#comment-13179608
 ] 

Bertrand Delacretaz commented on SLING-2347:


Whiteboard pattern implemented in revision 1227210

> Startup filter: allow requests to be rejected with a 503 during startup
> ---
>
> Key: SLING-2347
> URL: https://issues.apache.org/jira/browse/SLING-2347
> Project: Sling
>  Issue Type: New Feature
>  Components: Extensions
>Reporter: Bertrand Delacretaz
>Assignee: Bertrand Delacretaz
>Priority: Minor
>
> To provide a clear status during startup, especially when that implies 
> automatic application or content upgrades, I have implemented a simple filter 
> that causes all Sling requests to return a 503 status until another component 
> decides that Sling is ready to accept requests.
> The use case is to start the bundle that contains this filter at start level 
> lower than that of the Sling engine, so that the 503 filter kicks in as soon 
> as the Sling servlet starts, with a simple API to provide progress 
> information (returned in the body of the 503 response) and remove the filter 
> when the system is ready.
> I'll add that filter as a new bundle under contrib/extensions.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira




[jira] [Commented] (SLING-2340) jsp:forward no longer works after SLING-2094

2011-12-21 Thread Bertrand Delacretaz (Commented) (JIRA)

[ 
https://issues.apache.org/jira/browse/SLING-2340?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13174201#comment-13174201
 ] 

Bertrand Delacretaz commented on SLING-2340:


> Do we have some samples for forward?

or tests? ;-)

> jsp:forward no longer works after SLING-2094
> 
>
> Key: SLING-2340
> URL: https://issues.apache.org/jira/browse/SLING-2340
> Project: Sling
>  Issue Type: Improvement
>  Components: Scripting
>Affects Versions: Scripting JSP 2.0.18
>Reporter: Justin Edelson
>Priority: Critical
>
> The changes made for SLING-2094 seem to have completely broken support for 
> jsp:forward. Whereas the foward method in PageContextImpl() used to do a 
> forward, now it just throws an exception:
>  
> http://svn.apache.org/viewvc/sling/trunk/bundles/scripting/jsp/src/main/java/org/apache/sling/scripting/jsp/jasper/runtime/PageContextImpl.java?r1=1096843&r2=1145904

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira




[jira] [Commented] (SLING-2288) Bundle jar is silently ignored by the installer if MANIFEST.MF is not the first file in the archive

2011-11-15 Thread Bertrand Delacretaz (Commented) (JIRA)

[ 
https://issues.apache.org/jira/browse/SLING-2288?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13150597#comment-13150597
 ] 

Bertrand Delacretaz commented on SLING-2288:


The jar file spec doesn't seem to mention this, but the jar tool docs [1] 
indicates that jar always puts the manifest first in the jar.

https://issuetracker.springsource.com/browse/BNDLR-287 mentions that 
JarInputStream does enforce it while JarFile does not - I haven't checked that 
myself in the current JDKs.

So it looks like it's really a de facto requirement, and logging a warning is 
the right thing to do to avoid surprises.

[1] http://download.oracle.com/javase/1.3/docs/tooldocs/solaris/jar.html

> Bundle jar is silently ignored by the installer if MANIFEST.MF is not the 
> first file in the archive
> ---
>
> Key: SLING-2288
> URL: https://issues.apache.org/jira/browse/SLING-2288
> Project: Sling
>  Issue Type: Bug
>  Components: Installer
>Reporter: Bertrand Delacretaz
>Assignee: Carsten Ziegeler
>Priority: Minor
>
> A jar file where the MANIFEST.MF is not the first file in the jar is silently 
> ignored by the installer.
> We should at least output a warning, and maybe try harder to find the 
> manifest, in the Util.getManifest(...) method.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira




[jira] [Commented] (SLING-2288) Bundle jar is silently ignored by the installer if MANIFEST.MF is not the first file in the archive

2011-11-15 Thread Bertrand Delacretaz (Commented) (JIRA)

[ 
https://issues.apache.org/jira/browse/SLING-2288?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13150584#comment-13150584
 ] 

Bertrand Delacretaz commented on SLING-2288:


I'm absolutely fine with just logging - but if we can find the actual spec that 
helps explain what happens when people have to cope with broken jars.

> Bundle jar is silently ignored by the installer if MANIFEST.MF is not the 
> first file in the archive
> ---
>
> Key: SLING-2288
> URL: https://issues.apache.org/jira/browse/SLING-2288
> Project: Sling
>  Issue Type: Bug
>  Components: Installer
>Reporter: Bertrand Delacretaz
>Assignee: Carsten Ziegeler
>Priority: Minor
>
> A jar file where the MANIFEST.MF is not the first file in the jar is silently 
> ignored by the installer.
> We should at least output a warning, and maybe try harder to find the 
> manifest, in the Util.getManifest(...) method.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira




[jira] [Commented] (SLING-2288) Bundle jar is silently ignored by the installer if MANIFEST.MF is not the first file in the archive

2011-11-15 Thread Bertrand Delacretaz (Commented) (JIRA)

[ 
https://issues.apache.org/jira/browse/SLING-2288?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13150562#comment-13150562
 ] 

Bertrand Delacretaz commented on SLING-2288:


Do you have an URL to that spec? 

I see posts from usually knowledgeable people saying that having MANIFEST.MF 
first in a jar file is not required by the spec, so having that would be good 
;-)

> Bundle jar is silently ignored by the installer if MANIFEST.MF is not the 
> first file in the archive
> ---
>
> Key: SLING-2288
> URL: https://issues.apache.org/jira/browse/SLING-2288
> Project: Sling
>  Issue Type: Bug
>  Components: Installer
>Reporter: Bertrand Delacretaz
>Assignee: Carsten Ziegeler
>Priority: Minor
>
> A jar file where the MANIFEST.MF is not the first file in the jar is silently 
> ignored by the installer.
> We should at least output a warning, and maybe try harder to find the 
> manifest, in the Util.getManifest(...) method.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira




[jira] [Commented] (SLING-2254) ConcurrentModificationException in AdapterManagerImpl.createAdapterFactoryMap

2011-10-27 Thread Bertrand Delacretaz (Commented) (JIRA)

[ 
https://issues.apache.org/jira/browse/SLING-2254?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13136973#comment-13136973
 ] 

Bertrand Delacretaz commented on SLING-2254:


Patch looks good to me

> ConcurrentModificationException in AdapterManagerImpl.createAdapterFactoryMap
> -
>
> Key: SLING-2254
> URL: https://issues.apache.org/jira/browse/SLING-2254
> Project: Sling
>  Issue Type: Bug
>  Components: Extensions
>Affects Versions: Adapter 2.0.8
>Reporter: Bertrand Delacretaz
>Priority: Minor
> Attachments: FELIX-2254.patch
>
>
> Got this exception at least once during testing:
> org.apache.sling.engine.impl.SlingRequestProcessorImpl service: Uncaught 
> Throwable java.util.ConcurrentModificationException
>   at java.util.TreeMap$PrivateEntryIterator.nextEntry(TreeMap.java:1100)
>   at java.util.TreeMap$ValueIterator.next(TreeMap.java:1145)
>   at 
> org.apache.sling.adapter.internal.AdapterManagerImpl.createAdapterFactoryMap(AdapterManagerImpl.java:385)
>   at 
> org.apache.sling.adapter.internal.AdapterManagerImpl.getAdapterFactories(AdapterManagerImpl.java:354)
>   at 
> org.apache.sling.adapter.internal.AdapterManagerImpl.copyAdapterFactories(AdapterManagerImpl.java:432)
>   at 
> org.apache.sling.adapter.internal.AdapterManagerImpl.createAdapterFactoryMap(AdapterManagerImpl.java:401)
>   at 
> org.apache.sling.adapter.internal.AdapterManagerImpl.getAdapterFactories(AdapterManagerImpl.java:354)
>   at 
> org.apache.sling.adapter.internal.AdapterManagerImpl.copyAdapterFactories(AdapterManagerImpl.java:432)
>   at 
> org.apache.sling.adapter.internal.AdapterManagerImpl.createAdapterFactoryMap(AdapterManagerImpl.java:407)
>   at 
> org.apache.sling.adapter.internal.AdapterManagerImpl.getAdapterFactories(AdapterManagerImpl.java:354)
>   at 
> org.apache.sling.adapter.internal.AdapterManagerImpl.getAdapterFactories(AdapterManagerImpl.java:331)
>   at 
> org.apache.sling.adapter.internal.AdapterManagerImpl.getAdapter(AdapterManagerImpl.java:114)
>   at 
> org.apache.sling.api.adapter.SlingAdaptable.adaptTo(SlingAdaptable.java:82)

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira




[jira] [Commented] (SLING-2254) ConcurrentModificationException in AdapterManagerImpl.createAdapterFactoryMap

2011-10-25 Thread Bertrand Delacretaz (Commented) (JIRA)

[ 
https://issues.apache.org/jira/browse/SLING-2254?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13134983#comment-13134983
 ] 

Bertrand Delacretaz commented on SLING-2254:


The corresponding code is this itreation in createAdapterFactoryMap(...) :

if (afdMap != null) {
  for (AdapterFactoryDescriptor afd : afdMap.values()) {
...


> ConcurrentModificationException in AdapterManagerImpl.createAdapterFactoryMap
> -
>
> Key: SLING-2254
> URL: https://issues.apache.org/jira/browse/SLING-2254
> Project: Sling
>  Issue Type: Bug
>  Components: Extensions
>Affects Versions: Adapter 2.0.8
>Reporter: Bertrand Delacretaz
>Priority: Minor
>
> Got this exception at least once during testing:
> org.apache.sling.engine.impl.SlingRequestProcessorImpl service: Uncaught 
> Throwable java.util.ConcurrentModificationException
>   at java.util.TreeMap$PrivateEntryIterator.nextEntry(TreeMap.java:1100)
>   at java.util.TreeMap$ValueIterator.next(TreeMap.java:1145)
>   at 
> org.apache.sling.adapter.internal.AdapterManagerImpl.createAdapterFactoryMap(AdapterManagerImpl.java:385)
>   at 
> org.apache.sling.adapter.internal.AdapterManagerImpl.getAdapterFactories(AdapterManagerImpl.java:354)
>   at 
> org.apache.sling.adapter.internal.AdapterManagerImpl.copyAdapterFactories(AdapterManagerImpl.java:432)
>   at 
> org.apache.sling.adapter.internal.AdapterManagerImpl.createAdapterFactoryMap(AdapterManagerImpl.java:401)
>   at 
> org.apache.sling.adapter.internal.AdapterManagerImpl.getAdapterFactories(AdapterManagerImpl.java:354)
>   at 
> org.apache.sling.adapter.internal.AdapterManagerImpl.copyAdapterFactories(AdapterManagerImpl.java:432)
>   at 
> org.apache.sling.adapter.internal.AdapterManagerImpl.createAdapterFactoryMap(AdapterManagerImpl.java:407)
>   at 
> org.apache.sling.adapter.internal.AdapterManagerImpl.getAdapterFactories(AdapterManagerImpl.java:354)
>   at 
> org.apache.sling.adapter.internal.AdapterManagerImpl.getAdapterFactories(AdapterManagerImpl.java:331)
>   at 
> org.apache.sling.adapter.internal.AdapterManagerImpl.getAdapter(AdapterManagerImpl.java:114)
>   at 
> org.apache.sling.api.adapter.SlingAdaptable.adaptTo(SlingAdaptable.java:82)

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira




[jira] [Commented] (SLING-2245) Scala should use bundleized scala versions avialble in maven central

2011-10-14 Thread Bertrand Delacretaz (Commented) (JIRA)

[ 
https://issues.apache.org/jira/browse/SLING-2245?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13127458#comment-13127458
 ] 

Bertrand Delacretaz commented on SLING-2245:


@Michael thanks for the info, I changed it already. We should probably 
configure animal sniffer to ignore such warnings for scala, but we don't have 
much more of that in our codebase currently so nothing urgent.

> Scala should use bundleized scala versions avialble in maven central
> 
>
> Key: SLING-2245
> URL: https://issues.apache.org/jira/browse/SLING-2245
> Project: Sling
>  Issue Type: Improvement
>Reporter: Reto Bachmann-Gmür
>Assignee: Bertrand Delacretaz
> Attachments: SCALA-BUNDLE-VERSION-2.patch, 
> SCALA-BUNDLE-VERSION.patch, contrib-patch-fixed
>
>
> Currently it is using scala bundles from 
> http://scala-tools.org/repo-releases/, presumably equivalent artifacts are 
> avilable here: 
> http://repo2.maven.org/maven2/org/apache/servicemix/bundles/org.apache.servicemix.bundles.scala-compiler/

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira




[jira] [Commented] (SLING-2245) Scala should use bundleized scala versions avialble in maven central

2011-10-14 Thread Bertrand Delacretaz (Commented) (JIRA)

[ 
https://issues.apache.org/jira/browse/SLING-2245?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13127457#comment-13127457
 ] 

Bertrand Delacretaz commented on SLING-2245:


@Justin you're right, in this case animal sniffer catched the use of isEmpty() 
in compiled scala code, that's a good thing! Fixed in revision 1183298

> Scala should use bundleized scala versions avialble in maven central
> 
>
> Key: SLING-2245
> URL: https://issues.apache.org/jira/browse/SLING-2245
> Project: Sling
>  Issue Type: Improvement
>Reporter: Reto Bachmann-Gmür
>Assignee: Bertrand Delacretaz
> Attachments: SCALA-BUNDLE-VERSION-2.patch, 
> SCALA-BUNDLE-VERSION.patch, contrib-patch-fixed
>
>
> Currently it is using scala bundles from 
> http://scala-tools.org/repo-releases/, presumably equivalent artifacts are 
> avilable here: 
> http://repo2.maven.org/maven2/org/apache/servicemix/bundles/org.apache.servicemix.bundles.scala-compiler/

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira




[jira] [Commented] (SLING-2245) Scala should use bundleized scala versions avialble in maven central

2011-10-14 Thread Bertrand Delacretaz (Commented) (JIRA)

[ 
https://issues.apache.org/jira/browse/SLING-2245?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13127432#comment-13127432
 ] 

Bertrand Delacretaz commented on SLING-2245:


I'm trying the contrib-patch-fixed, and the build fails with an animalSniffer 
error due to this line of scala code in 
script/src/main/scala/org/apache/sling/scripting/scala/BundleFS.scala

lazy val fullName: String = (path::name::Nil).filter(!_.isEmpty).mkString("/")

> Scala should use bundleized scala versions avialble in maven central
> 
>
> Key: SLING-2245
> URL: https://issues.apache.org/jira/browse/SLING-2245
> Project: Sling
>  Issue Type: Improvement
>Reporter: Reto Bachmann-Gmür
> Attachments: SCALA-BUNDLE-VERSION-2.patch, 
> SCALA-BUNDLE-VERSION.patch, contrib-patch-fixed
>
>
> Currently it is using scala bundles from 
> http://scala-tools.org/repo-releases/, presumably equivalent artifacts are 
> avilable here: 
> http://repo2.maven.org/maven2/org/apache/servicemix/bundles/org.apache.servicemix.bundles.scala-compiler/

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira




[jira] [Commented] (SLING-2192) Support JAX-RS resource classes

2011-10-12 Thread Bertrand Delacretaz (Commented) (JIRA)

[ 
https://issues.apache.org/jira/browse/SLING-2192?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13125683#comment-13125683
 ] 

Bertrand Delacretaz commented on SLING-2192:


Agreed, so that would be contrib/extensions/jaxrs, with integration tests under 
contrib/launchpad/testing

> Support JAX-RS resource classes
> ---
>
> Key: SLING-2192
> URL: https://issues.apache.org/jira/browse/SLING-2192
> Project: Sling
>  Issue Type: New Feature
>Reporter: Reto Bachmann-Gmür
> Attachments: SLING-2192-20110310.patch, SLING-2192-20111004.patch, 
> SLING-2192-new-jax-rs-bundle.patch, SLING-2192-new-jax-rs-bundle.patch, 
> SLING-2192-with-sling-style-style-registration.patch, 
> SLING-2192-with-tests.patch
>
>
> It should be possible to register jax resource classes and providers as 
> services. As they don't implement a specific interface services that expose 
> java.lang.Object should be considered as javx-rs services iff they have the 
> service property "javax.ws.rs" set to true.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira




[jira] [Commented] (SLING-2192) Support JAX-RS resource classes

2011-10-12 Thread Bertrand Delacretaz (Commented) (JIRA)

[ 
https://issues.apache.org/jira/browse/SLING-2192?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13125665#comment-13125665
 ] 

Bertrand Delacretaz commented on SLING-2192:


Thanks for the updated patch, I still have a few comments:

1) Are the RequestUriOptingServlet2Test.java and RequestUriOptingServlet2 tests 
useful with your latest patch? IIUC you switched to servlet filters so they're 
probably not needed, or should be part of a separate issue.

2) The SimpleRootResourceTest of my SLING-2192-with-tests patch is missing

3) The new bundle should be at bundles/extensions/jaxrs like in that patch

4) I think we don't need the samples/slingrs, we'll create integration tests 
instead, that also act as code samples

> Support JAX-RS resource classes
> ---
>
> Key: SLING-2192
> URL: https://issues.apache.org/jira/browse/SLING-2192
> Project: Sling
>  Issue Type: New Feature
>Reporter: Reto Bachmann-Gmür
> Attachments: SLING-2192-20110310.patch, SLING-2192-20111004.patch, 
> SLING-2192-new-jax-rs-bundle.patch, SLING-2192-new-jax-rs-bundle.patch, 
> SLING-2192-with-sling-style-style-registration.patch, 
> SLING-2192-with-tests.patch
>
>
> It should be possible to register jax resource classes and providers as 
> services. As they don't implement a specific interface services that expose 
> java.lang.Object should be considered as javx-rs services iff they have the 
> service property "javax.ws.rs" set to true.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira