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

Fixed link to the installation instructions #1841

Merged
merged 2 commits into from
Feb 27, 2014
Merged

Conversation

pSub
Copy link
Member

@pSub pSub commented Feb 26, 2014

The fragment identifier was wrong/outdated.

@domenkozar
Copy link
Member

We should probably give it a name to prevent such problems in the future (see in manual source how there are defined)

@vcunat
Copy link
Member

vcunat commented Feb 26, 2014

I agree, this change would just postpone the problem again.

@pSub
Copy link
Member Author

pSub commented Feb 27, 2014

Makes sense, I have added a static identifier to that chapter and adapted the link. I also tested it locally using nix-build -A config.system.build.manual.manual and it worked.

@vcunat
Copy link
Member

vcunat commented Feb 27, 2014

Looks good to me.

domenkozar added a commit that referenced this pull request Feb 27, 2014
Fixed link to the installation instructions
@domenkozar domenkozar merged commit 4e957b0 into NixOS:master Feb 27, 2014
@pSub pSub deleted the patch-1 branch February 27, 2014 14:10
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