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

Fixes #9382: Change the Rudder version in the sources to be substituted in the packaging script #1111

Conversation

amousset
Copy link
Member

@@ -60,6 +61,9 @@ localdepends: ./rudder-sources ./maven/bin/mvn ./rudder-users.xml ./rudder-doc .
tar -xjf rudder-sources.tar.bz2
mv rudder-sources-*/ rudder-sources/

# Change rudder version displayed in the Web UI
sed -i "s%PACKAGING_RUDDER_VERSION%Rudder ${RUDDER_VERSION_TO_PACKAGE}%" rudder-sources/${HOMEPAGEPATH}
Copy link
Member

Choose a reason for hiding this comment

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

We should have a kind of tag delimiter like < > or &xx&

Moreover we should replace the tag with the version only, because if we want to change the text we should only have to modify the original html file.

@amousset amousset force-pushed the ust_9382/change_the_rudder_version_in_the_sources_to_be_substituted_in_the_packaging_script branch from 783e6d7 to b918a08 Compare October 17, 2016 13:41
@amousset
Copy link
Member Author

Commit modified

1 similar comment
@amousset
Copy link
Member Author

Commit modified

@amousset amousset force-pushed the ust_9382/change_the_rudder_version_in_the_sources_to_be_substituted_in_the_packaging_script branch from b918a08 to 19b1042 Compare October 17, 2016 13:41
@amousset
Copy link
Member Author

Commit modified

@amousset amousset force-pushed the ust_9382/change_the_rudder_version_in_the_sources_to_be_substituted_in_the_packaging_script branch from 19b1042 to 57c27c4 Compare October 17, 2016 13:51
@amousset
Copy link
Member Author

Commit modified

@amousset amousset force-pushed the ust_9382/change_the_rudder_version_in_the_sources_to_be_substituted_in_the_packaging_script branch from 57c27c4 to ffb9610 Compare October 17, 2016 13:52
@Normation-Quality-Assistant
Copy link
Contributor

OK, merging this PR

@Normation-Quality-Assistant Normation-Quality-Assistant merged commit ffb9610 into Normation:master Oct 17, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants