Skip to content

Adding initial sample project - #27

Open
KevinSchildhorn wants to merge 7 commits into
mainfrom
AddingSampleProject
Open

Adding initial sample project#27
KevinSchildhorn wants to merge 7 commits into
mainfrom
AddingSampleProject

Conversation

@KevinSchildhorn

Copy link
Copy Markdown
Contributor

Adding small sample project that adds monocypher, a crypto library.

Base automatically changed from UpdatingReadme to main August 12, 2026 17:08
Comment thread sample/README.md Outdated
}
```

* **`mingwX64` is deliberately not declared: `secure_random.c` has no Windows branch, and adding one

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bold markdown isn't closed so it displays as asterisks. Stray > on line 44

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Updated

@@ -0,0 +1,7 @@
distributionBase=GRADLE_USER_HOME

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

gradle-wrapper.properties, but no gradlew binary for the sample

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Updated

Comment thread sample/build.gradle.kts Outdated

nativeTargets.forEach { target ->
target.compilations.getByName("main").cinterops.create("monocypher") {
defFile(project.file("src/nativeInterop/cinterop/monocypher.def"))

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think this line is required if the def file name matches the create name above

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good catch I removed it

Comment thread .gitignore Outdated
.gradle/
build/

# Kotlin

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fun fact AI caught. This isn't a comment. The # has to be the first character of the line and there is a space here. So it's actually a functioning rule which ignores a file named # Kotlin

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

huh interesting, ok I updated those comments

@@ -0,0 +1,151 @@
/*
* Copyright (c) 2021 Touchlab

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Update copyright on all files

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

updated

@KevinSchildhorn

Copy link
Copy Markdown
Contributor Author

I also updated the kotlin and gradle versions that were causing build issues with newer versions of Android Studio and JDK

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.

2 participants