X-Recipient: archive-cygwin AT delorie DOT com
DomainKey-Signature: a=rsa-sha1; c=nofws; d=sourceware.org; h=list-id
	:list-unsubscribe:list-subscribe:list-archive:list-post
	:list-help:sender:date:from:reply-to:message-id:to:subject
	:in-reply-to:references:mime-version:content-type
	:content-transfer-encoding; q=dns; s=default; b=lYfi79DE4lz6/4k/
	x+sXB0EjLXZmRBgQu4v7JEoTpXjlzTqfTYLRoFr4agy1NMqZH7eWCBiFk5d2rrWr
	P+F7/qUZT9VPqDsDRmYgE5prh9agjQ9NDcu86qo6R+ChJqi4SIQc6c48skoquV4T
	e78gJMNWEpEePbEeORi9VALGiRg=
DKIM-Signature: v=1; a=rsa-sha1; c=relaxed; d=sourceware.org; h=list-id
	:list-unsubscribe:list-subscribe:list-archive:list-post
	:list-help:sender:date:from:reply-to:message-id:to:subject
	:in-reply-to:references:mime-version:content-type
	:content-transfer-encoding; s=default; bh=lV+k3vzDRXkjrMzrCxOv8a
	xjGPk=; b=VVDesdk3OKJIJYqzS0TlKIE0dfQ9B8TFmRydSo2KYorkq/RyFppor4
	4Lw3N2EL49ld5ucLkBb1Fuv+9t0IvyyIKgoh3nz2TiPwWruTwladRDbdsLgf+WcS
	gCoddfQX5xp2gVTtDjENWYaWPcvM1jnq6jyCZcUmzVR1joigbcO/o=
Mailing-List: contact cygwin-help AT cygwin DOT com; run by ezmlm
List-Id: <cygwin.cygwin.com>
List-Subscribe: <mailto:cygwin-subscribe AT cygwin DOT com>
List-Archive: <http://sourceware.org/ml/cygwin/>
List-Post: <mailto:cygwin AT cygwin DOT com>
List-Help: <mailto:cygwin-help AT cygwin DOT com>, <http://sourceware.org/ml/#faqs>
Sender: cygwin-owner AT cygwin DOT com
Mail-Followup-To: cygwin AT cygwin DOT com
Delivered-To: mailing list cygwin AT cygwin DOT com
Authentication-Results: sourceware.org; auth=none
X-Spam-SWARE-Status: No, score=-0.7 required=5.0 tests=BAYES_00,FREEMAIL_FROM,KAM_THEBAT,RCVD_IN_DNSWL_LOW,SPF_PASS autolearn=no version=3.3.1 spammy=UD:ru, H*UA:Bat!, doubts, H*R:U*cygwin
X-HELO: forward105o.mail.yandex.net
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=yandex.ru; s=mail; t=1558893003;	bh=nO+SWJZpzTcFJZunvsWktbnLNIrJvQoB9+b/DW8hMBU=;	h=In-Reply-To:Subject:To:Reply-To:From:Message-ID:References:Date;	b=wcMyU0mm0mj8S6sPrZmhsqIKT+vftRvI5azT88eWO7GM9syrmgKM0m63hATJ19t+J	 8zdnqhOFFPYIU2KxyCNXbLIQzlUz7rOO+d2LxVjZz0cN1+NRPCCLLpkddf+WSJ8XlP	 k06bhFCz1NnqkDsvyQqNnwkZBdWS2arn1BTj6wBM=
Authentication-Results: mxback15j.mail.yandex.net; dkim=pass header.i=@yandex.ru
Date: Sun, 26 May 2019 20:47:25 +0300
From: Andrey Repin <anrdaemon AT yandex DOT ru>
Reply-To: cygwin AT cygwin DOT com
Message-ID: <1812401761.20190526204725@yandex.ru>
To: Sorin Adrian Savu <sorin AT itguy DOT ro>, cygwin AT cygwin DOT com
Subject: Re: openssl-devel in openssh README
In-Reply-To: <CAFeE7Gkavqhw+WWL_17B5fSWADBD9VKxW0tw0XY+9Nb+6OxBQA@mail.gmail.com>
References: <CAFeE7G=n-uqwVJDgeYAjZ9qu+jK7cQ2ZEZv9gR-fHa3Pg-CXNw AT mail DOT gmail DOT com>   <120396920 DOT 20190526103526 AT yandex DOT ru>  <CAFeE7Gkavqhw+WWL_17B5fSWADBD9VKxW0tw0XY+9Nb+6OxBQA AT mail DOT gmail DOT com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Transfer-Encoding: 7bit
X-IsSubscribed: yes

Greetings, Sorin Adrian Savu!

>> Greetings, Sorin Adrian Savu!
>>
>> > Hello!
>>
>> > I had to compile openssh recently, and the README for cygwin is
>> referencing
>> > openssl-devel package which is obsolete now. I spent some time trying to
>> > figure this out. Can you please update
>> >
>> https://github.com/openssh/openssh-portable/blob/85ceb0e64bff672558fc87958cd548f135c83cdd/contrib/cygwin/README#L80
>> > to
>> > mention libssl-devel instead ?
>>
>> > (I would do it, but I can't figure out how to submit a PR to that repo)
>>
>> Clone the repo on github to your account, create branch, make changes,
>> submit
>> a PR.


> As a generic approach, that would do, however openssh-portable has quite a
> few branches and being a security related package I was expecting some
> strict guidelines on how to submit patches (which I can't find).

If there's any guidelines, a bot would mail you once you submit a PR, like it
is so for Symfony subpackages. (When you file a PR against them, a bot comes
in, closes it and tells you to fu... erm, file it against main Symfony repo.)

> Anyhow, I see that all the patches start from master, so I will give it a
> try.

Why I suggest a branch is because somebody could more easily inspect it
locally, when there's doubts.
Checking out from remote's master is a little bit more convoluted and
error-prone, than from a branch.


-- 
With best regards,
Andrey Repin
Sunday, May 26, 2019 20:43:59

Sorry for my terrible english...


--
Problem reports:       http://cygwin.com/problems.html
FAQ:                   http://cygwin.com/faq/
Documentation:         http://cygwin.com/docs.html
Unsubscribe info:      http://cygwin.com/ml/#unsubscribe-simple