Skip to content

Conversation

DongHan2016
Copy link
Member

  1. Moved the folder OracleTSAM to the folder OracleTuxedo and rename OracleTSAM to tsam. There is no change in the contents of Dockerfiles in OracleTSAM.
  2. Update OracleTuxedo/README.md for the new sub-folder

removed the useless file
Updated following files:
core/dockerfiles/12.1.3/Dockerfile
core/dockerfiles/12.2.2/Dockerfile
core/dockerfiles/buildDockerImage.sh
core/dockerfiles/README.md
core/README.md
core/samples/mp/docker-compose.yml
core/samples/mp/Dockerfile
core/samples/mp/simpappmp_runme.sh
core/samples/shm/build.sh
core/samples/shm/Dockerfile
core/samples/shm/simpapp_runme.sh

Additions:
core/dockerfiles/12.1.3: init.sh
core/dockerfiles/12.2.2: init.sh
core/samples/mp: simpappmp_build.sh

Removals:
core/dockerfiles/12.1.3: tuxedo12.1.3.rsp.template
core/dockerfiles/12.1.3: tuxedo12.1.3_silent_install.sh.template
core/dockerfiles: fix_locations.sh
core/dockerfiles: tuxedo.docker
1. removed core/samples to tuxedo_full as the samples are dependent on
TSAM which is not installed in the docker
2. removed dockerfiles and samples folders under the original Tuxedo
folder (OracleTuxedo)
3. updated the files according to the new comments in PR455.
1. Moved the folder OracleTSAM to OracleTuxedo and renamed it to tsam,
and without any changes in contents of those files.
2. Updated OracleTuxedo/README.md for the new sub-folder tsam.
Djelibeybi
Djelibeybi previously approved these changes Jul 17, 2017
@Djelibeybi
Copy link
Contributor

I've approved this, but in future, please be more careful with your branches: there are a lot of commits in here that have already been merged. You need to rebase your master branch from the upstream master and then create a local branch for your changes before submitting a pull request.

I recommend the Git and GitHub learning resources here: https://help.github.com/articles/git-and-github-learning-resources/

Particularly the bits on branching, forks and pull requests.

@Djelibeybi Djelibeybi merged commit 550d688 into oracle:master Jul 17, 2017
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.

3 participants