Skip to content

Konwersja do nowego csproj oraz użycie nugetów#1

Open
bartcho wants to merge 20 commits intomasterfrom
soneta-nuget
Open

Konwersja do nowego csproj oraz użycie nugetów#1
bartcho wants to merge 20 commits intomasterfrom
soneta-nuget

Conversation

@bartcho
Copy link
Copy Markdown

@bartcho bartcho commented Oct 10, 2018

No description provided.

@bartcho bartcho closed this Oct 10, 2018
@bartcho bartcho reopened this Oct 10, 2018
@bartcho
Copy link
Copy Markdown
Author

bartcho commented Oct 10, 2018

      To będzie problem jak ktoś pobierze ten przykład. Bo nie uda mu się pobrać pakietów. Chyba, że gdzieś jest info i przeoczyłem. Dobrze by było mieć feeda publicznego lub na nuget.org.

Ten PR będzie ciągnięty do wydania oficjalnego pakietów i wtedy NuGet.Config wyleci, bo pakiety będą opublikowane w normalnym publicznym feed'zie. Na ten moment publikowanie tych pakietów publicznie nawet nie ma sensu.

@TomekScieplek
Copy link
Copy Markdown
Contributor

Tak się zastanawiam czy dobrze się stało, że wersję nugeta określa się w Directory.Build.props. Wcześniej propsy zrobiłem też w MSBuild\common.soneta.assembly.props i może dobrze byłoby to połączyć (obojętnie w którą stronę)?

@TomekScieplek
Copy link
Copy Markdown
Contributor

W sumie to zmodyfikuję też te swoje propsy, bo potrzebne może być już tylko jedno property: ExtensionOtput. SonetaBinaries nie ma już racji bytu.

@bartcho
Copy link
Copy Markdown
Author

bartcho commented Oct 15, 2018

@TomekScieplek Directory.Build.props ma pewną charakterystyczną cechę, dołączane jest do wszystkich projektów. Daje nam to taką zaletę, że nie musimy robić ręcznych importów, a w zależności od zastosowania nawet w ogóle zmieniać plików projektów.
Co do tego w jakiej konfiguracji i czy w ogóle Output projektu dodatku powinien być ustawiany na katalog, z którego domyślnie ładujemy dodatki to zupełnie inny temat. Wg mnie powinien być inny sposób na załadowanie dodatku niż pchanie go do katalogu, który wymaga uprawnień administracyjnych.

@TomekScieplek
Copy link
Copy Markdown
Contributor

Co do tego w jakiej konfiguracji i czy w ogóle Output projektu dodatku powinien być ustawiany na katalog, z którego domyślnie ładujemy dodatki to zupełnie inny temat. Wg mnie powinien być inny sposób na załadowanie dodatku niż pchanie go do katalogu, który wymaga uprawnień administracyjnych.

Zgadza się i powiem szczerze, że zastanawiałem się nad tym, żeby w README opisać sposób na budowę solucji ze wskazaniem outputu z linii komend, natomiast pomimo ułomności rozwiązania, ze względu na wygodę partnerów na razie zostawiłem ExtensionOutput.

@TomekScieplek
Copy link
Copy Markdown
Contributor

Po przejściu na nowe SDK jawne referencje do Soneta.Products.Test i Soneta.Products.Modules nie będą już potrzebne.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants