> On Jan. 25, 2017, 5:41 p.m., haosdent huang wrote:
> > Do you forgot to update `src/Makefile.am`
> > 
> > ```
> > nobase_dist_pkgdata_DATA +=                                         \
> >   webui/master/static/js/angular-1.2.3.js                           \
> >   webui/master/static/js/angular-1.2.3.min.js                               
> > \
> >   webui/master/static/js/angular-route-1.2.3.js                             
> > \
> >   webui/master/static/js/angular-route-1.2.3.min.js                 \
> >   webui/master/static/js/jquery-1.7.1.js                            \
> >   webui/master/static/js/jquery-1.7.1.min.js                                
> > \
> >   webui/master/static/js/relative-date.js                           \
> >   webui/master/static/js/ui-bootstrap-tpls-0.9.0.js                 \
> >   webui/master/static/js/ui-bootstrap-tpls-0.9.0.min.js                     
> > \
> >   webui/master/static/js/underscore-1.4.3.js                                
> > \
> >   webui/master/static/js/underscore-1.4.3.min.js                    \
> >   webui/master/static/js/zeroclipboard-1.1.7.js                             
> > \
> >   webui/master/static/js/zeroclipboard-1.1.7.min.js                 \
> >   webui/master/static/obj/zeroclipboard-1.1.7.swf
> > 
> > ```

I have fixed this when submit it, thank you for the contribution!


- haosdent


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/55791/#review162975
-----------------------------------------------------------


On Jan. 22, 2017, 6:11 p.m., Jacob Janco wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/55791/
> -----------------------------------------------------------
> 
> (Updated Jan. 22, 2017, 6:11 p.m.)
> 
> 
> Review request for mesos and haosdent huang.
> 
> 
> Bugs: MESOS-6969
>     https://issues.apache.org/jira/browse/MESOS-6969
> 
> 
> Repository: mesos
> 
> 
> Description
> -------
> 
> - Use pure js clipboard.js and remove
>   flash dependency.
> - Vendored zeroclipboard has security issues
>   https://www.cvedetails.com/cve/CVE-2014-1869/
> 
> 
> Diffs
> -----
> 
>   src/webui/master/static/css/mesos.css 
> c817aaee0fd2d2579ff2e4a152d47e83ce80d68c 
>   src/webui/master/static/index.html a4eda3034906d100a20dcd0a2473112c1edd6b42 
>   src/webui/master/static/js/app.js c32177aa23c962f2bdf03d98272fb6d21a565382 
>   src/webui/master/static/js/clipboard-1.5.16.js PRE-CREATION 
>   src/webui/master/static/js/clipboard-1.5.16.min.js PRE-CREATION 
>   src/webui/master/static/js/zeroclipboard-1.1.7.js 
> d45551d6c9f75c275032219768549f76865b9cd3 
>   src/webui/master/static/js/zeroclipboard-1.1.7.min.js 
> 32535fddf1baab255fcbf5d3d776a0a9ac1447e8 
>   src/webui/master/static/obj/zeroclipboard-1.1.7.swf 
> 880e64ee7614e224660c6616b4bbb1ee5fab8ec9 
> 
> Diff: https://reviews.apache.org/r/55791/diff/
> 
> 
> Testing
> -------
> 
> make check + verification of clipboard functionality
> 
> 
> Thanks,
> 
> Jacob Janco
> 
>

Reply via email to