openssl-devel in openssh README

openssl-devel in openssh README

[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)

Hope this helps!

Sorin Savu

--
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

Re: openssl-devel in openssh README

[Andrey Repin]

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.


-- 
With best regards,
Andrey Repin
Sunday, May 26, 2019 10:33:37

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

Re: openssl-devel in openssh README

[Brian Inglis]

On 2019-05-26 00:19, Sorin Adrian Savu wrote:
> 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 ?

Obsolete package openssl-devel now depends on libssl-devel, if using the current
setup, so you get the correct packages installed and used: look at the package
setup.hint or the setup.ini entry.
Cygwin obsolete packages are most similar to other distros virtual packages, but
they can require specific dependencies and/or versions be installed: this avoids
updating dependencies and docs just because dependency package names change.

-- 
Take care. Thanks, Brian Inglis, Calgary, Alberta, Canada

This email may be disturbing to some readers as it contains
too much technical detail. Reader discretion is advised.

--
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

Re: openssl-devel in openssh README

[Sorin Adrian Savu]

On Sun, May 26, 2019 at 10:50 AM Andrey Repin <anrdaemon@yandex.ru> wrote:

> 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).
Anyhow, I see that all the patches start from master, so I will give it a
try.

>
>

> --
> With best regards,
> Andrey Repin
> Sunday, May 26, 2019 10:33:37
>
> 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

Re: openssl-devel in openssh README

[Andrey Repin]

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