[ 
https://issues.apache.org/jira/browse/BIGTOP-1193?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13880391#comment-13880391
 ] 

Sean Mackrory commented on BIGTOP-1193:
---------------------------------------

I wouldn't have thought so, now that you mention it, but I don't know why that 
would ever be needed, so I can't say with confidence it isn't needed. That was 
also added by BIGTOP-1150. Any insight [~rvs]?

> Update Spark packaging for various conventions
> ----------------------------------------------
>
>                 Key: BIGTOP-1193
>                 URL: https://issues.apache.org/jira/browse/BIGTOP-1193
>             Project: Bigtop
>          Issue Type: Bug
>            Reporter: Sean Mackrory
>            Assignee: Sean Mackrory
>         Attachments: 
> 0001-BIGTOP-1193.-Update-Spark-packaging-for-various-conv.patch
>
>
> This is a bit of a mixed bag of things I noticed:
> * We're using the old style of checking multiple locations for bigtop-utils. 
> If we depend on bigtop-utils 0.7 we can safely assume where it is.
> * We're invoking install_spark.sh with `sh` instead of `bash` in one place - 
> someone's going to get a cryptic syntax error one day :)
> * We're invoking do-component-build in deb/.../rules with an unnecessary 
> environment variable (accidentally introduced in BIGTOP-1150 - it's a 
> downstream modification specific to a Certain Distribution of Hadoop based on 
> Bigtop).



--
This message was sent by Atlassian JIRA
(v6.1.5#6160)

Reply via email to