Skip to content
This repository has been archived by the owner on Sep 1, 2022. It is now read-only.

Added dependency list for Fedora #1005

Closed
wants to merge 2 commits into from

Conversation

shmacoshmuesday
Copy link


By submitting this pull-request, I confirm the following:

  • I have read and understood the developer guide in kovri-docs.
  • I have checked that another pull-request for this purpose does not exist.
  • I have considered and confirmed that this submission will be valuable to others.
  • I accept that this submission may not be used and that this pull-request may be closed by the will of the maintainer.
  • I give this submission freely under the BSD 3-clause license.

Copy link
Contributor

@coneiric coneiric left a comment

Choose a reason for hiding this comment

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

Change formatting and content looks good, thanks for the contribution.

I'm not sure how many flavors of Linux we want to display dependencies for, though.

Copy link
Collaborator

@anonimal anonimal left a comment

Choose a reason for hiding this comment

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

Hi @shmacoshmuesday. Please $ git rebase -i master and fixup the most recent commit so we only have 1 commit to merge. Thanks.

@anonimal
Copy link
Collaborator

anonimal commented Sep 6, 2018

Oh, that's assuming you were using a topic branch, which you aren't. You can try something like $ git rebase -i HEAD~2 instead.

@anonimal
Copy link
Collaborator

anonimal commented Sep 7, 2018

NOTICE: THIS PULL REQUEST HAS BEEN MOVED TO GitLab. Please reopen there if you wish to merge. See #1013 for details.

@anonimal anonimal closed this Sep 7, 2018
@shmacoshmuesday
Copy link
Author

Sorry for my late response, and thanks for the instructions anonimal. I'll checkout gitlab and submit again, though it may take a while simply because I have a lot of things competing for my time most of the time. Thanks for all you do anonimal and coneiric!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants