
<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.0//EN" "http://www.w3.org/TR/REC-html40/strict.dtd">

<html><head><meta name="qrichtext" content="1" /><style type="text/css">

p, li { white-space: pre-wrap; }

</style></head><body style=" font-family:'Oxygen-Sans'; font-size:10pt; font-weight:400; font-style:normal;">

<p style=" margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;">Hello,</p>

<p style="-qt-paragraph-type:empty; margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; "> </p>

<p style=" margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px;">I'm proposing some changes to the Dockerfile.</p>

<p style=" margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px;">Considering the Docker guides, it is best to split the initialization into multiple RUN commands so that caching can be used.</p>

<p style=" margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px;">I've done that and some other improvements and commited them to a branch of the androidbuilder repo, since we don't have this repo in a review platform.</p>

<p style="-qt-paragraph-type:empty; margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; "> </p>

<p style=" margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px;">Branch is here: <a href="https://quickgit.kde.org/?p=scratch%2Fcordlandwehr%2Fandroidbuilder.git&a=shortlog&h=a259aa7fef4849f70970a8e9fa82f8c467747e9d"><span style=" text-decoration: underline; color:#2980b9;">https://quickgit.kde.org/?p=scratch%2Fcordlandwehr%2Fandroidbuilder.git&a=shortlog&h=a259aa7fef4849f70970a8e9fa82f8c467747e9d</span></a></p>

<p style="-qt-paragraph-type:empty; margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; "> </p>

<p style=" margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px;">Please provide feedback.<br />-- </p>

<p style=" margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"><span style=" font-family:'oxygen mono'; font-size:1.2em; font-weight:600;">Ovidiu-Florin Bogdan</span></p>

<p style=" margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"><a href="http://ovidiu.geekaliens.com"><span style=" font-family:'oxygen mono'; text-decoration: underline; color:#2980b9;">GeekAliens.com</span></a><span style=" font-family:'oxygen mono';"> </span></p>

<p style=" margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"><a href="http://kubuntu.org"><span style=" font-family:'oxygen mono'; text-decoration: underline; color:#2980b9;">Kubuntu Romania Representative</span></a></p>

<p style=" margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"><a href="http://kubuntu.org"><span style=" font-family:'oxygen mono'; text-decoration: underline; color:#2980b9;">Kubuntu Council Member</span></a></p></body></html>