use openssl instead of sha256sum for build.rs #568

Birleştirildi

Plume_migration_agent openssl içindeki 2 işlemeyi master ile birleştirdi

trinity-1686a yorum yaptı

Sahibi

fix #566

igalic (github.com konumundan göç edildi) incelendi

igalic (github.com konumundan göç edildi) bir yorum yaptı

👀

build.rs

igalic (github.com konumundan göç edildi) yorum yaptı

i'd prefer we leave stat(1) out of this, as the --printf option is not platform independent, and no equivalent, platform independent option exists

the only other would be to use -c

i'd prefer we leave `stat(1)` out of this, as the `--printf` option is not platform independent, and no equivalent, platform independent option exists the only other would be to use `-c`

trinity-1686a incelendi

build.rs

trinity-1686a yorum yaptı

Poster

Sahibi

The goal of this function is to have a unique id, and best effort is made to not invalidate cache if not necessary. We can also not make that best effort (recompiling Plume => new cache id) and just generate a random hex string. That would be platform independent.
What shall I do?

codecov[bot] yorum yaptı (github.com konumundan göç edildi)

Codecov Report

Merging #568 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #568   +/-   ##
=======================================
  Coverage   35.06%   35.06%           
=======================================
  Files          67       67           
  Lines        7851     7851           
  Branches     1886     1886           
=======================================
  Hits         2753     2753           
  Misses       4336     4336           
  Partials      762      762

# [Codecov](https://codecov.io/gh/Plume-org/Plume/pull/568?src=pr&el=h1) Report > Merging [#568](https://codecov.io/gh/Plume-org/Plume/pull/568?src=pr&el=desc) into [master](https://codecov.io/gh/Plume-org/Plume/commit/33619abdfbe1845f56d283265bae2e3e68ad0485?src=pr&el=desc) will **not change** coverage. > The diff coverage is `n/a`. ```diff @@ Coverage Diff @@ ## master #568 +/- ## ======================================= Coverage 35.06% 35.06% ======================================= Files 67 67 Lines 7851 7851 Branches 1886 1886 ======================================= Hits 2753 2753 Misses 4336 4336 Partials 762 762 ```

igalic (github.com konumundan göç edildi) incelendi

build.rs

igalic (github.com konumundan göç edildi) yorum yaptı

i think i'd just go with stat -c for now

there's not a big chance we'll be building on windows… easily… i think, so let's just go with this.

i think i'd just go with `stat -c` for now there's not a big chance we'll be building on windows… easily… i think, so let's just go with this.

igalic (github.com konumundan göç edildi) bu değişiklikleri onayladı

igalic (github.com konumundan göç edildi) bir yorum yaptı

tested, works!

Gözden Geçirenler

Değişiklik isteği 5b50f90d2b olarak birleştirildi.

komut satırı talimatlarını da görüntüleyebilirsiniz.

1. Adım:

Proje deponuzdan yeni bir dala göz atın ve değişiklikleri test edin.

git checkout -b openssl master

git pull origin openssl

2. Adım:

Forgejo'daki değişiklikleri ve güncellemeleri birleştirin.

git checkout master

git merge --no-ff openssl

git push origin master

Bu konuşmaya katılmak için oturum aç.

Değerlendirici yok

Kilometre Taşı Yok

Atanan Kişi Yok

2 Katılımcı

Bildirimler

Bitiş Tarihi

Bitiş tarihi geçersiz veya aralık dışında. Lütfen 'yyyy-aa-gg' biçimini kullanın.

Bitiş tarihi atanmadı.

Bağımlılıklar

Bağımlılık yok.

Yükleniyor…

Henüz bir içerik yok.

Etiketler Kilometre Taşları

use openssl instead of sha256sum for build.rs #568

Codecov Report

Gözden Geçirenler

1. Adım:

2. Adım: