New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add strongname support #178

Merged
merged 2 commits into from Jan 31, 2019

Conversation

Projects
None yet
3 participants
@sisiplac
Copy link
Contributor

sisiplac commented Jan 25, 2019

No description provided.

@sisiplac sisiplac requested a review from f00f Jan 25, 2019

@mischastik
Copy link
Contributor

mischastik left a comment

Der Pfad zum Key zeigt in ein absolutes Verzeichnis. Das ist nicht optimal, weil man das GitHub Projekt nicht direkt bauen kann. Er sollte wahrscheinlich auf das Projektverzeichnis zeigen, aber man sollte den Key selber wahrscheinlich eher nicht Einchecken.

@sisiplac

This comment has been minimized.

Copy link
Contributor Author

sisiplac commented Jan 27, 2019

Der Pfad zum Key zeigt in ein absolutes Verzeichnis. Das ist nicht optimal, weil man das GitHub Projekt nicht direkt bauen kann. Er sollte wahrscheinlich auf das Projektverzeichnis zeigen, aber man sollte den Key selber wahrscheinlich eher nicht Einchecken.

Unseren Metrilus-Key sollten wir auf jeden Fall nicht mit einchecken, man könnte höchstens über einen eigenen Key für MetriCam2 nachdenken und diesen dann mit ins Repository legen. Im Grunde besteht aber bei all unseren Kamera-Implementierungen das gleiche Problem (Verweise auf Z:), weshalb ich es erstmal so lassen würde - es sei denn, es gäbe einen besseren Alternativvorschlag.

Show resolved Hide resolved BetaCameras/Kinect2/Kinect2.csproj
Show resolved Hide resolved BetaCameras/OrbbecOpenNI/OrbbecOpenNI.vcxproj
Show resolved Hide resolved BetaCameras/OrbbecOpenNI/OrbbecOpenNI.vcxproj Outdated
Show resolved Hide resolved BetaCameras/OrbbecOpenNI/OrbbecOpenNI.vcxproj Outdated
Show resolved Hide resolved BetaCameras/OrbbecOpenNI/OrbbecOpenNI.vcxproj Outdated
Show resolved Hide resolved BetaCameras/RealSense/RealSense.csproj
Show resolved Hide resolved BetaCameras/RealSense2/RealSense2.NetStandard.csproj Outdated
Show resolved Hide resolved MetriCam2_SDK.sln
Show resolved Hide resolved Scripts/Jenkinsfile.groovy Outdated

@f00f f00f force-pushed the add-strongname-support branch from 96a96dd to 60bf33a Jan 30, 2019

@f00f

This comment has been minimized.

Copy link
Contributor

f00f commented Jan 30, 2019

All my issues are resolved. Before merging please squash all commits but 9e4b5a3

@f00f

f00f approved these changes Jan 30, 2019

@sisiplac sisiplac force-pushed the add-strongname-support branch from 0fa2ce1 to 3ec3018 Jan 30, 2019

@mischastik mischastik merged commit 989ad92 into master Jan 31, 2019

1 check passed

MetriCam2 CI Build successful
Details

@mischastik mischastik deleted the add-strongname-support branch Jan 31, 2019

@f00f

This comment has been minimized.

Copy link
Contributor

f00f commented Feb 5, 2019

Hm, it would have been nice if not all commits had been squashed, as expressed in #178 (comment)

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