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

Add immae repository #132

Merged
merged 1 commit into from
May 24, 2019
Merged

Add immae repository #132

merged 1 commit into from
May 24, 2019

Conversation

immae
Copy link
Contributor

@immae immae commented May 2, 2019

Note: I get an error error: opening lock file '/nix/var/nix/db/big-lock': Permission denied when running ./bin/nur update (from several repositories, not only mine), so I couldn’t run the checks.


The following points apply when adding a new repository to repos.json

  • I ran nur/format-manifest.py after updating repos.json (We will use the same script in travis ci to make sure we keep the format consistent)
  • By including this repository in NUR I give permission to license the
    content under the MIT license.

Clarification where license should apply:
The license above does not apply to the packages built by the
Nix Packages collection, merely to the package descriptions (i.e., Nix
expressions, build scripts, etc.). It also might not apply to patches
included in Nixpkgs, which may be derivative works of the packages to
which they apply. The aforementioned artifacts are all covered by the
licenses of the respective packages.

@Mic92
Copy link
Member

Mic92 commented May 2, 2019

Would it be possible to split of the packages or not have the nixops repository that contains unrelated content (i.e. websites)? We like to keep the size over all repositories reasonable for https://github.com/nix-community/nur-combined

@immae
Copy link
Contributor Author

immae commented May 3, 2019

@Mic92 : I’m not against the idea of splitting it, but my priority is to ensure usability and simplicity for (myself and) potential collaborators of my hosting project, so I don’t know if I’ll find a suitable solution for your request (I’m open to suggestions though, I’m still learning).

Note that I tried hard to make everything outside of the nixops dir as unspecific as possible, and there are still things in there that are currently being moved into the parent and are of interest to more people (like peertube, mastodon, diaspora).

@Mic92
Copy link
Member

Mic92 commented May 3, 2019

I think @makefu had some script to automatically separate packages from https://git.shackspace.de/krebs/stockholm into https://github.com/makefu/nur-packages

@immae
Copy link
Contributor Author

immae commented May 3, 2019

Ah, right, I can play with subtrees (but that will make the update manual only), I’ll keep you updated

@makefu
Copy link
Member

makefu commented May 3, 2019

@immae check out the ci file i created for only publishing the subtree:
https://git.shackspace.de/krebs/stockholm/blob/master/.gitlab-ci.yml

cheers!

@immae
Copy link
Contributor Author

immae commented May 3, 2019

Thanks @makefu I’ll do something with it (not immediately, but at least now I have a solution :) )

@Mic92 Mic92 mentioned this pull request May 11, 2019
@immae
Copy link
Contributor Author

immae commented May 23, 2019

@Mic92 I finally finished to cleanup my repository: I separated "private" files from "public" ones and applied @makefu suggestion of filter-branch to remove the private ones from history (my script also squashes all history until "today", to reduce the initial size)

One of my package made Travis fail (due to a configuration that forbids derivation inputs determined by another derivation, allow-import-from-derivation), so I remove it during the filter-branch.

Now Travis is happy for my repo :)

@Mic92 Mic92 merged commit 6a00fc2 into nix-community:master May 24, 2019
@Mic92
Copy link
Member

Mic92 commented May 24, 2019

Thanks!

milahu pushed a commit to milahu/NUR that referenced this pull request Aug 5, 2023
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.

3 participants