Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

FedoraLdp [PREVIEW] #492

Closed
wants to merge 30 commits into from
Closed

FedoraLdp [PREVIEW] #492

wants to merge 30 commits into from

Conversation

cbeer
Copy link
Contributor

@cbeer cbeer commented Sep 25, 2014

No description provided.

* See the License for the specific language governing permissions and
* limitations under the License.
*/
package org.fcrepo.http.api;
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some day, we should move this to a new module for just LDP things (and make -http-api required it).

Andrew Woods and others added 6 commits September 26, 2014 11:15
Updated Modeshape to Beta2, which required mock object updates
The specific functionality that's causing the symptom noted in PT # 79728226 was introduced via commit 068c346

Rationale for the PR:

ReadOnlyExternalPropertiesFedoraFileSystemConnectorIT fails when trying to delete a file generated when running
BasicReadWriteFedoraFileSystemConnectorIT. The former makes no use of this file, and hence shouldn't try to delete it (indirectly via AbstractFedoraFileSystemConnectorIT).

This fixes the specific build error that result from attempting to delete a file that's still held on by another Java process. The file lock was observed on Windows Java 7u67 (and 7u25). I used the Windows handle command to see that the file was indeed being held open by a java process. I was unable to get further details about which thread was holding the file open. As this seems to be a common problem on Windows, it's not clear to me whether this is worth investigating.
@cbeer cbeer force-pushed the ldp-resource branch 2 times, most recently from f9b7f49 to 4ffb2be Compare October 2, 2014 15:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants