Re: [DISCUSS] Graduate Apache Gobblin (Incubating) as a TLP

2021-01-02 Thread Abhishek Tiwari
Hi Justin,

For 1: I have removed the remaining logos from the "Execution Modes"
section on homepage
For 2: I have removed unreleased tags from docker (which were all the tags
published so far)

Please review.

Abhishek

On Sat, Jan 2, 2021 at 6:51 PM Abhishek Tiwari  wrote:

> Hi Justin,
>
> Thanks for the review again.
>
> 1. I see that we missed the logos under "Execution Modes", I will remove
> those as well.
>
> 4. Regarding docker, per policy [1] I thought unreleased tags are fine
> (ie. if its nightly or snapshot), where-as the *latest* tag should point
> to an (approved) released code. Seems like my understanding was incorrect,
> so I will remove the unreleased tags from there as well.
>
> [1] https://incubator.apache.org/guides/distribution.html#docker
>
> Abhishek
>
> On Sat, Jan 2, 2021 at 6:30 PM Justin Mclean 
> wrote:
>
>> HI,
>>
>> > 1. 3rd party logos being eaten by pacman on the homepage: We have
>> removed
>> > this animation, and created a ticket to follow-up with Trademarks for
>> > future usage of that animation.
>>
>> There are still trademarks on that page that are not mentioned in the
>> footer disclaimer. You might also have to look at [1][2][3][4] on what you
>> can and cannot do with these 3rd party trademarks.
>>
>> > 4. Docker not following guidelines: We have deleted the pre-apache repo
>> (
>> > https://hub.docker.com/r/gobblin ), and updated the apache/gobblin on
>> > DockerHub to bring it inline with the guidelines.
>>
>> It looks like you may have unrealised code available via docker? [5]
>> Please note this [6] from the distribution policy. In particular unreleased
>> materials must not be distributed through channels which encourage use by
>> anyone outside the project development community.
>>
>> Thanks,
>> Justin
>>
>> 1. https://www.oracle.com/legal/trademarks.html
>> 2. https://www.oracle.com/legal/logos.html
>> 3.
>> http://www.oracle.com/us/technologies/java/java-licensing-logo-guidelines-1908204.pdf
>> 4. https://aws.amazon.com/trademark-guidelines/
>> 5.
>> https://hub.docker.com/r/apache/gobblin/tags?page=1=last_updated
>> 6. https://infra.apache.org/release-distribution.html#unreleased
>>
>>
>>
>>
>>
>>
>>
>>
>> -
>> To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org
>> For additional commands, e-mail: general-h...@incubator.apache.org
>>
>>


Re: Link to a page that seems to be missing

2021-01-02 Thread Justin Mclean
Hi,

Here is the last version of that page (from 2017):
https://web.archive.org/web/20170629200901/http://incubator.apache.org/facilities.html

I can see facilities.xml in the old Incubator/content  content area but no 
facilities.ad in the new area. Perhaps it was not converted? I couldn’t find 
any discussion about this, does anyone remember why it may have been removed?

Thanks,
Justin



-
To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org
For additional commands, e-mail: general-h...@incubator.apache.org



[GitHub] [incubator] justinmclean commented on pull request #89: Update faq.ad

2021-01-02 Thread GitBox


justinmclean commented on pull request #89:
URL: https://github.com/apache/incubator/pull/89#issuecomment-753560035


   LGTM (although we should probably update some of this as it's quite old)



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



-
To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org
For additional commands, e-mail: general-h...@incubator.apache.org



Re: [DISCUSS] Graduate Apache Gobblin (Incubating) as a TLP

2021-01-02 Thread Abhishek Tiwari
Hi Justin,

Thanks for the review again.

1. I see that we missed the logos under "Execution Modes", I will remove
those as well.

4. Regarding docker, per policy [1] I thought unreleased tags are fine (ie.
if its nightly or snapshot), where-as the *latest* tag should point to an
(approved) released code. Seems like my understanding was incorrect, so I
will remove the unreleased tags from there as well.

[1] https://incubator.apache.org/guides/distribution.html#docker

Abhishek

On Sat, Jan 2, 2021 at 6:30 PM Justin Mclean 
wrote:

> HI,
>
> > 1. 3rd party logos being eaten by pacman on the homepage: We have removed
> > this animation, and created a ticket to follow-up with Trademarks for
> > future usage of that animation.
>
> There are still trademarks on that page that are not mentioned in the
> footer disclaimer. You might also have to look at [1][2][3][4] on what you
> can and cannot do with these 3rd party trademarks.
>
> > 4. Docker not following guidelines: We have deleted the pre-apache repo (
> > https://hub.docker.com/r/gobblin ), and updated the apache/gobblin on
> > DockerHub to bring it inline with the guidelines.
>
> It looks like you may have unrealised code available via docker? [5]
> Please note this [6] from the distribution policy. In particular unreleased
> materials must not be distributed through channels which encourage use by
> anyone outside the project development community.
>
> Thanks,
> Justin
>
> 1. https://www.oracle.com/legal/trademarks.html
> 2. https://www.oracle.com/legal/logos.html
> 3.
> http://www.oracle.com/us/technologies/java/java-licensing-logo-guidelines-1908204.pdf
> 4. https://aws.amazon.com/trademark-guidelines/
> 5.
> https://hub.docker.com/r/apache/gobblin/tags?page=1=last_updated
> 6. https://infra.apache.org/release-distribution.html#unreleased
>
>
>
>
>
>
>
>
> -
> To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org
> For additional commands, e-mail: general-h...@incubator.apache.org
>
>


Re: [DISCUSS] Graduate Apache Gobblin (Incubating) as a TLP

2021-01-02 Thread Justin Mclean
HI,

> 1. 3rd party logos being eaten by pacman on the homepage: We have removed
> this animation, and created a ticket to follow-up with Trademarks for
> future usage of that animation.

There are still trademarks on that page that are not mentioned in the footer 
disclaimer. You might also have to look at [1][2][3][4] on what you can and 
cannot do with these 3rd party trademarks.

> 4. Docker not following guidelines: We have deleted the pre-apache repo (
> https://hub.docker.com/r/gobblin ), and updated the apache/gobblin on
> DockerHub to bring it inline with the guidelines.

It looks like you may have unrealised code available via docker? [5] Please 
note this [6] from the distribution policy. In particular unreleased materials 
must not be distributed through channels which encourage use by anyone outside 
the project development community.

Thanks,
Justin

1. https://www.oracle.com/legal/trademarks.html
2. https://www.oracle.com/legal/logos.html
3. 
http://www.oracle.com/us/technologies/java/java-licensing-logo-guidelines-1908204.pdf
4. https://aws.amazon.com/trademark-guidelines/
5. https://hub.docker.com/r/apache/gobblin/tags?page=1=last_updated
6. https://infra.apache.org/release-distribution.html#unreleased








-
To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org
For additional commands, e-mail: general-h...@incubator.apache.org



Re: [VOTE] Graduate Apache Gobblin (Incubating) as a TLP

2021-01-02 Thread Abhishek Tiwari
Hi Justin,

We have tried to address and resolve the issues that you had identified,
please review the discuss thread: https://s.apache.org/4zde4
and let us know if it looks ok to you?

Thanks,
Abhishek

On Wed, Dec 23, 2020 at 5:15 PM Justin Mclean 
wrote:

> Hi,
>
> -1 (binding) due to docker and trademark issues, but happy to change my
> vote to +1 once those are on teh way to bee corrected.
>
> Thanks,
> Justin
> -
> To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org
> For additional commands, e-mail: general-h...@incubator.apache.org
>
>


Re: NOTICE file for binary release

2021-01-02 Thread Antoine Toulme
Understood, thanks for taking a look. I was worried about something obvious I’d 
miss before I get to run a release candidate by the IPMC.

I will start a release vote now.

Cheers,

Antoine

> On Jan 2, 2021, at 6:00 PM, Justin Mclean  wrote:
> 
> Hi,
> 
> Without access to a release it is a little hard to confirm, but it looks 
> reasonable to me.
> 
> Thanks,
> Justin
> -
> To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org
> For additional commands, e-mail: general-h...@incubator.apache.org
> 


-
To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org
For additional commands, e-mail: general-h...@incubator.apache.org



Re: NOTICE file for binary release

2021-01-02 Thread Justin Mclean
Hi,

Without access to a release it is a little hard to confirm, but it looks 
reasonable to me.

Thanks,
Justin
-
To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org
For additional commands, e-mail: general-h...@incubator.apache.org



Re: [VOTE] Release Apache Hop (incubating) 0.50-rc2

2021-01-02 Thread Justin Mclean
HI,

Sorry but it’s -1 (binding) as it contains Category B source code and header 
issues and is not using the WIP disclaimer. [1]

 I suggest you use the work in progress disclaimer, and in it list the issues 
that need to be fixed before graduation. Fixing some of teh header issues would 
make it easier to review as well.

I checked:
- Incubating in name
- DISCLAIMER exists but is not work in progress 
- LICENSE is missing mention of 3rd party included software [2][3][5][6]
- NOTICE may need updating if the release contains code from Apache Hadoop
- There a large number of files that are missing ASF headers. This makes it 
hard to review.
- There are a large number of files without correct headers copyright  Hitachi 
Vantara.
- Files incorrectly have the ASF header onto of anther license header e.g [4]
- Includes Category B course code
- No unexpected binary files
- Unable to compile on OSX but not sure if that’s a supported platform. It gets 
to step 467 out of 474. (Hop Assemblies Plugins Dist)

Thanks,
Justin

1. 
https://incubator.apache.org/policy/incubation.html#work_in_progress_disclaimer
2. OpenSansLight font
3. 
./plugins/actions/ftp/src/main/java/org/apache/hop/workflow/actions/ftpput/FtpClient.java
4. 
./ui/src/main/java/org/apache/hop/ui/pipeline/transforms/tableinput/SqlValuesHighlight.java
5. ./assemblies/static/src/main/resources/LICENSE ( Copyright (c) 2005, 
European Commission project)
6 ./assemblies/static/src/main/resources/docs/English/welcome/css/bootstrap.css
-
To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org
For additional commands, e-mail: general-h...@incubator.apache.org



Re: [VOTE] Release Apache Hop (incubating) 0.50-rc2

2021-01-02 Thread Justin Mclean
Hi,

> Hop plans to fix these issues in a later incubating release. I hope that 
> exceptions to release policy can be made for these and similar issues in this 
> early-stage incubating release.

If you have used the work in progress DISCLAIMER and listed the potential 
issues in that file this is fine.

Thanks,
Justin
-
To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org
For additional commands, e-mail: general-h...@incubator.apache.org