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

Problem with overwriting existing folder when exporting to SIARD-DK #480

Closed
T33py opened this issue Apr 8, 2021 · 5 comments
Closed

Problem with overwriting existing folder when exporting to SIARD-DK #480

T33py opened this issue Apr 8, 2021 · 5 comments
Assignees
Labels
bug Issue type
Milestone

Comments

@T33py
Copy link

T33py commented Apr 8, 2021

Description:
When running the following command:
java "-Dfile.encoding=UTF-8" -jar dbptk-app-2.9.7.jar migrate -i "microsoft-sql-server" -is "DB" -idb "ArkiveringsTestDatabase" -iu "dbptk" -ip "pwd123" -ide -il -e "siard-dk" -ef "D:\Projekter\projekt\DBPTK\tst2\AVID.SA.12223.1"

with the folder 'D:\Projekter\projekt\DBPTK\tst2\AVID.SA.12223.1' present on the disk the program produces this output:

Converting database: microsoft-sql-server to siard-dk Initializing database Exporting SIARD version dk ERROR Fatal error while converting the database (Error deleting existing directory) Log files and migration reports were saved in D:\Pojekter\Jammerbugt\DBPTK\Program Troubleshooting information can be found at https://github.com/keeps/dbptk-developer/wiki/Troubleshooting Please report any problems at https://github.com/keeps/dbptk-developer/issues/new

This was not resolved by running in admin mode for some reason, and the issue is quite severe in the gui version, since you can't target folders that doesn't exist there.

The exception is thrown from the file dbptk-developer\dbptk-modules\dbptk-module-siard\src\main\java\com\databasepreservation\modules\siard\out\output\SIARDDKDatabaseExportModule.java between lines 68 and 75. This is also shown in the log.

Steps required to reproduce the bug:

  1. Create target folder (empty)
  2. Run dbptk-app-2.9.7.jar with the newly created target folder and SIARD-DK as the chosen output

dbptk-app.log.txt

Attach the dbptk-app.log.txt file below.

@T33py
Copy link
Author

T33py commented Apr 8, 2021

I should probably have mentioned that I ran the program on windows.

Also I would suspect that the call basicFileAttributes.creationTime().toString(); at line 71 may result in an invalid foldername.

@hmiguim hmiguim self-assigned this Apr 8, 2021
@hmiguim hmiguim added the bug Issue type label Apr 8, 2021
@hmiguim
Copy link
Member

hmiguim commented Apr 8, 2021

@T33py in the next days I shall look into this bug and report with a possible solution

@aatrisgn
Copy link

I am experiencing the same issue with a fresh installation of the desktop database-preservation application (On windows).

Is there any work-around for this bug by using the console application instead?

@aatrisgn
Copy link

It seems like there is a work-around for this initial problem for the desktop application. If you create a given folder, choose it via the desktop application and delete the folder after creation, the application can continue.

@T33py
Copy link
Author

T33py commented Jun 4, 2021

I am experiencing the same issue with a fresh installation of the desktop database-preservation application (On windows).

Is there any work-around for this bug by using the console application instead?

The console application does not have this bug. It seems to be a difference in the setup before calling the library used by both.

hmiguim added a commit that referenced this issue Jun 8, 2021
@hmiguim hmiguim closed this as completed in e36f302 Jun 8, 2021
@hmiguim hmiguim added this to the 2.9.8 milestone Jun 8, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Issue type
Projects
None yet
Development

No branches or pull requests

3 participants