Skip to content
This repository was archived by the owner on Dec 5, 2021. It is now read-only.

Layer update (to 108)#980

Closed
knocte wants to merge 46 commits intosochix:masterfrom
nblockchain:layerUpdate
Closed

Layer update (to 108)#980
knocte wants to merge 46 commits intosochix:masterfrom
nblockchain:layerUpdate

Conversation

@knocte
Copy link
Collaborator

@knocte knocte commented Sep 11, 2020

Supersedes nblockchain#5

knocte and others added 30 commits April 2, 2020 15:55
Now @aarani takes over as main contributor and the possible
receiver of donations, since sochix hasn't contributed in a
long time.
Otherwise if there's a PR whose build is failing, it would show
a red badge here (but we only care about master branch).

Spotted & fix by @aarani
This pipeline doesn't come with Mono out-of-the-box like the GH-Actions
one, so then we need to install mono ourselves (and there are 3
different ways to do it).
WARNING: NU5118: File 'D:\a\TLSharp\TLSharp\TLSharp.Core\bin\Debug\TLSharp.Core.dll' is not added because the package already contains file 'lib\net45\TLSharp.Core.dll'

(This commit was brought from TLSharp [1] version 0.1.0.521.)

[1] sochix@6b35857
Because we might use dependencies soon that require this framework.
And anyway these days the .NET world is mostly a hybrid between
.NETStandard/.NETCore libs and .NET4.6 ones, not older than that.

(This commit was brought from TLSharp [1] version 0.1.0.522.)

[1] sochix@709e5be
(This commit was brought from TLSharp [1] version 0.1.0.524.)

[1] sochix@6d2c77f
Fixes sochix#857

(This commit was brought from TLSharp [1] version 0.1.0.532.)

[1] sochix@c67801a
Thanks to some inspiration by this comment:
sochix#474 (comment)
Fixes sochix#935

(This commit was brought from TLSharp [1] version 0.1.0.554.)

[1] sochix@9a6e391
It was only meant to be used internally.
The schema is in https://core.telegram.org/schema/json so it's more
logical to expect the file to be named `json`, which is what would
happen if you download it with wget, for example.
Core: allow reconnecting same TelegramClient instance
It was hardcoding the Windows path separator so this could only
run on Windows before.
Bear with me, the part of the code that traverses the path with ".."
subfolders is actually not needed, but whatever, I had already written
it.
Somehow for the previous commit hash, which had only numbers in its
short version (0122072), nuget failed with:
Run $date = get-date -format "yyyyMMdd-HHmm"
Attempting to build package from 'TgSharp.Core.csproj'.
Error NU5010: Version string specified for package reference '0.1.0-date.20200414-0908.git.0122072' is invalid.
File does not exist (TgSharp.0.1.0-date.20200414-0908.git.0122072.nupkg).

Now we remove the "." after "date" and "git" and use double-dash
to separate initial version from the sub-version.
This makes Layer updates much easier, without having to
include new files manually.
Now that we have diverged from TLSharp[1], the new pre-release
versions pushed to nuget have different API and slightly
different behaviour so we want them to be marked as newer.

[1] since 4d09ac7
Otherwise if the hash starts with a non-number it looks weird.
@knocte
Copy link
Collaborator Author

knocte commented Sep 11, 2020

Ooops, wrong.

@knocte knocte closed this Sep 11, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants