Skip to content
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

feat(build): #546.1 packages #549

Merged
merged 1 commit into from
Sep 16, 2021
Merged

feat(build): #546.1 packages #549

merged 1 commit into from
Sep 16, 2021

Conversation

kamadorueda
Copy link
Contributor

  • Expose everything as packages, pending to see
    how we can identify apps

@kamadorueda
Copy link
Contributor Author

@blaggacao

Example project:

diff --git a/flake.nix b/flake.nix
new file mode 100644
index 0000000..77b8b78
--- /dev/null
+++ b/flake.nix
@@ -0,0 +1,15 @@
+{
+  description = "A Makes example";
+
+  inputs = {
+    flakeUtils.url = "github:numtide/flake-utils";
+    makes.url = "/data/github/kamadorueda/makes";
+  };
+
+  outputs = { flakeUtils, makes, ... } @ inputs:
+    flakeUtils.lib.eachDefaultSystem
+      (system: makes.lib.fromFlake {
+        inherit inputs;
+        inherit system;
+      });
+}
diff --git a/makes.nix b/makes.nix
new file mode 100644
index 0000000..6b8fcb8
--- /dev/null
+++ b/makes.nix
@@ -0,0 +1,5 @@
+{
+  helloWorld = {
+    enable = true;
+    name = "test";
+  };
+}

image

image

- Expose everything as packages, pending to see
  how we can identify apps
@blaggacao
Copy link
Contributor

Very interesting, give me some time to review in-depth. Does this mean we loose the m wrapper?

In a sense I find this pretty useful, since it let's me easily override / adapt some things and also the interface is simpler.

The following would be ideal:

  • plain nix interface for the power user
  • m for the normal dev guys

@kamadorueda
Copy link
Contributor Author

@blaggacao Everything has a first version and the good thing here is that it proofs that it's possible!

@kamadorueda kamadorueda merged commit 1e50a46 into fluidattacks:main Sep 16, 2021
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