Skip to content
This repository has been archived by the owner on Aug 16, 2023. It is now read-only.

[IMP] odoo-shippable: Load .bash_aliases at the end of .bashrc #230

Merged
merged 1 commit into from
Jan 7, 2018

Conversation

luisg123v
Copy link
Contributor

Currently, when the Docker image docker-odoo-image is built up, all modifications to the file .bashrc are appended at the end of the file.

Even though this behavior works just fine in most cases, it also overrides possible-user-defined settings on the file ~/.bash_aliases, because that file is loaded before all modifications, thus making image settings taking precedence over user-defined settings.

This change re-locates the inclusion of .bash_aliases at the end of ~/.bashrc, so all settings and aliases defined on .bash_aliases are not overridden anymore.

@luisg123v
Copy link
Contributor Author

I should note that, even though Travis tests are not passing, that issue is not related to my modifications and it comes from before, as reported on #214

@luisg123v
Copy link
Contributor Author

@moylop260
I noticed this is still pending. Could you merge this, please?

@moylop260 moylop260 merged commit bd4052b into Vauxoo:master Jan 7, 2018
@moylop260 moylop260 deleted the master-bash_aliases-luisg branch January 7, 2018 02:30
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants