[ 
http://jira.dspace.org/jira/browse/DS-297?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=10521#action_10521
 ] 

Mark Diggory commented on DS-297:
---------------------------------

InitializeDatabase already lookes in the appropriate directory. I believe you 
are not setting your db.name appropriately in the dspace.cfg you are running 
your ant build against.

See: 
https://scm.dspace.org/svn/repo/dspace/trunk/dspace-api/src/main/java/org/dspace/storage/rdbms/InitializeDatabase.java

    /**
     * Attempt to get the named script, with the following rules:
     * etc/<db.name>/<name>
     * etc/<name>
     * <name>
     */
    private static FileReader getScript(String name) throws 
FileNotFoundException, IOException
    {
        String dbName = ConfigurationManager.getProperty("db.name");
        File myFile = null;
        
        if (dbName != null)
        {
            myFile = new File("etc/" + dbName + "/" + name);
            if (myFile.exists())
                return new FileReader(myFile.getCanonicalPath());
        }
        
        myFile = new File("etc/" + name);
        if (myFile.exists())
            return new FileReader(myFile.getCanonicalPath());
        
        return new FileReader(name);
    }


Mark




> Refactor SQL source and Ant script to avoid copying Oracle versions over 
> PostgreSQL
> -----------------------------------------------------------------------------------
>
>                 Key: DS-297
>                 URL: http://jira.dspace.org/jira/browse/DS-297
>             Project: DSpace 1.x
>          Issue Type: Improvement
>          Components: Documentation, DSpace API
>    Affects Versions: 1.6.0
>         Environment: n/a
>            Reporter: Larry Stone
>            Priority: Minor
>
> The current installation procedure for Oracle requires the Oracle SQL files 
> to be _copied over_ the PostgreSQL versions; this is a kludgy modification to 
> the source code hierarchy that renders your local copy different from the 
> prototype, e.g. in svn.  This is an unnecessary and gratuitous difference, 
> and it also adds a confusing step to the installation just for Oracle users, 
> who are already likely to be more confused than their luckier PostgreSQL 
> colleagues. 
> I propose putting the SQL files for Oracle, PostgreSQL, and whatever other 
> DBs may someday get supported, side-by-side in peer directories, and have 
> them used in place without any copying.  The only change to the source tree 
> would be to add dspace/etc/postgres and move all the "default" SQL files in 
> dspace/etc to there.  Then, the build.xml script would simply insert the 
> value of the "db.name" configuration property (ant can read properties 
> easily)  into the path, or modify org.dspace.storage.rdbms.InitializeDatabase 
> to do so.  There would no longer be any need to modify the source tree as 
> part of an Oracle installation. 
> I'm willing to do this and update the documentation, for 1.6.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: 
http://jira.dspace.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

------------------------------------------------------------------------------
Let Crystal Reports handle the reporting - Free Crystal Reports 2008 30-Day 
trial. Simplify your report design, integration and deployment - and focus on 
what you do best, core application coding. Discover what's new with 
Crystal Reports now.  http://p.sf.net/sfu/bobj-july
_______________________________________________
Dspace-devel mailing list
Dspace-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dspace-devel

Reply via email to