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

[MXNET-250] CSS and .htaccess fix #10286

Merged
merged 13 commits into from
Apr 10, 2018

Conversation

thinksanky
Copy link
Contributor

Description

[ ] Copy .htaccess from master to root folder.
[ ] Copy CSS fix (mxnet.css) into all the versions.
[ ] Test out link - http://34.207.78.24/

@thinksanky thinksanky requested a review from szha as a code owner March 27, 2018 22:54
@thinksanky thinksanky changed the title Css and htaccess fix [MXNET-250] CSS and .htaccess fix Mar 29, 2018
@aaronmarkham
Copy link
Contributor

aaronmarkham commented Apr 7, 2018

We need to have the package links in there, but this will be handled with a subsequent PR.

Copy link
Member

@eric-haibin-lin eric-haibin-lin left a comment

Choose a reason for hiding this comment

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

Some minor comments

@@ -68,6 +68,7 @@ for tag in $tag_list; do
git checkout master -- docs/mxdoc.py
fi
git submodule update || exit 1
git submodule update --init --recursive
Copy link
Member

Choose a reason for hiding this comment

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

Why not update line 70?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I have tried updating it together, but its unstable. I have found this to be better and working consistently

# During a nightly build, these fixes will be patched to all the versions in the asf-site repository including the master folder under versions directory.
# copy <master folder location> <version folder location>

#SIM: https://sim.amazon.com/issues/AWSDBUX-19727
Copy link
Member

Choose a reason for hiding this comment

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

I don't think we should put public unaccessible link here

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

python AddPackageLink.py --file_path "$built/versions/master/install/index.html" \
--current_version "$tag" || exit 1
fi
#if [ $tag != 'master' ]
Copy link
Member

Choose a reason for hiding this comment

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

Remove unused code?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

@eric-haibin-lin eric-haibin-lin merged commit 6db9dd4 into apache:master Apr 10, 2018
rahul003 pushed a commit to rahul003/mxnet that referenced this pull request Jun 4, 2018
* css_and_install_fixes

* fixed the broken installation page

* copy install index.html fix to 1.1.0 and master versions

* fixed install.md

* fixed variable definition and install copy logic

* added .htaccess and updated .htaccess copy to root folder

* copy .htaccess to root folder

* updated update_all_version.sh

* fixed renaming of the function call

* reverted back AddVersion.py from master

* removed duplicate master being appended

* commented out AddPackageLink.py to fix the master install page issue

* removed commented out lines and not required
zheng-da pushed a commit to zheng-da/incubator-mxnet that referenced this pull request Jun 28, 2018
* css_and_install_fixes

* fixed the broken installation page

* copy install index.html fix to 1.1.0 and master versions

* fixed install.md

* fixed variable definition and install copy logic

* added .htaccess and updated .htaccess copy to root folder

* copy .htaccess to root folder

* updated update_all_version.sh

* fixed renaming of the function call

* reverted back AddVersion.py from master

* removed duplicate master being appended

* commented out AddPackageLink.py to fix the master install page issue

* removed commented out lines and not required
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants