Skip to content

Conversation

@bamcgill
Copy link
Contributor

Updated files for builds for 23c, sqldeveloper, sqlcl, ORDS a…nd APEX with demo schemas installed

@bamcgill bamcgill requested a review from gvenzl January 16, 2026 12:25
@bamcgill bamcgill self-assigned this Jan 16, 2026
@oracle-contributor-agreement oracle-contributor-agreement bot added the OCA Verified All contributors have signed the Oracle Contributor Agreement. label Jan 16, 2026
Copy link

@gvenzl gvenzl left a comment

Choose a reason for hiding this comment

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

There are still many 23c references, but that has long been superseded by 26ai. Should we not update this to that?

@bamcgill
Copy link
Contributor Author

bamcgill commented Jan 19, 2026

Reviewed 26 changes, removed 23 references. Thanks @gvenzl

@bamcgill
Copy link
Contributor Author

There are still many 23c references, but that has long been superseded by 26ai. Should we not update this to that?

fixed

Copy link

@gvenzl gvenzl left a comment

Choose a reason for hiding this comment

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

I found a couple of more things that need a second look.

@@ -0,0 +1,30 @@
#!/bin/bash
Copy link

Choose a reason for hiding this comment

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

Should be renamed to installDatabaseFree.sh

set -x

. ~oracle/runTimeStartScript.sh
(/tmp/1/3_2installDatabaseXE_inner.sh "$@" 2>&1) | tee -a ~oracle/log/thestdoutlog_`basename $0`_$$ | egrep '^PROGRESS'
Copy link

Choose a reason for hiding this comment

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

should be renamed to ...DatabaseFree_inner.sh

@@ -0,0 +1,106 @@
#!/bin/bash
Copy link

Choose a reason for hiding this comment

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

Should be renamed to ...DatabaseFree_inner.sh


#################################################################################
#
# File: 3_2installDatabaseXE.sh
Copy link

Choose a reason for hiding this comment

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

should be renamed

export TWO_TASK=
' > /tmp/env1
chmod 755 /tmp/env1
cp /etc/sysconfig/oracle-xe-18c.conf ~oracle
Copy link

Choose a reason for hiding this comment

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

18c XE?

@@ -0,0 +1,57 @@

REM
REM Copyright (c) 2017, Oracle and/or its affiliates. All rights reserved.
Copy link

Choose a reason for hiding this comment

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

2017 copyright?

# Do not change the following system generated value.
#-------------------------------------------------------------------------------
oracle.install.responseFileVersion=/oracle/install/rspfmt_dbinstall_response_schema_v18.0.0
oracle.install.responseFileVersion=/oracle/install/rspfmt_dbinstall_response_schema_v19.0.0
Copy link

Choose a reason for hiding this comment

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

19c?

@@ -0,0 +1,591 @@
rem
Copy link

Choose a reason for hiding this comment

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

wouldn't it be better to pull this from the web and install either the latest or a stable version instead of redistributing?

Just a question, I have nothing against it.

@@ -1,4 +1,4 @@
#!/bin/sh
bin/sh
Copy link

Choose a reason for hiding this comment

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

Seems like a typo. It should probably not be there or not have been modified

3topLevel.sh PROGRESS: 3/4 3topLevel.sh - main kick off script i.e. yum and call subscripts
3_1installDbtoolClientTools.sh PROGRESS: 3_1/4 3_1installDbtoolClientTools.sh - install dbtool client tools
3_2installDatabase.sh PROGRESS: 3_2/4 3_2installDatabase.sh - install the database
3_2installDatabaseXE.sh PROGRESS: 3_2/4 3_2installDatabaseXE.sh - install the database if XE
Copy link

Choose a reason for hiding this comment

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

Should be renamed to ...DatabaseFree etc.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

OCA Verified All contributors have signed the Oracle Contributor Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants