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.

Sign up for GitHub

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

Jump to bottom

podspell script: set output encoding from locale LC_CTYPE #17

Merged
merged 1 commit into from Feb 25, 2015
Merged

podspell script: set output encoding from locale LC_CTYPE #17

merged 1 commit into from Feb 25, 2015

Conversation

dolmen
Copy link
Contributor

@dolmen dolmen commented Feb 23, 2015

Fix the podspell to use the encoding from the locale.

Signed-off-by: Olivier Mengué dolmen@cpan.org

@dolmen
podspell script: set output encoding from locale LC_CTYPE
11d8c47 
Signed-off-by: Olivier Mengué <dolmen@cpan.org>
xenoterracide added a commit that referenced this pull request Feb 25, 2015
@xenoterracide
Merge pull request #17 from dolmen/set-locale-encoding
9b056ec 
podspell script: set output encoding from locale LC_CTYPE
@xenoterracide xenoterracide merged commit 9b056ec into perl-pod:master Feb 25, 2015
@dolmen
Copy link
Contributor Author

dolmen commented Sep 1, 2015

This patch has been reverted in #18 and must be applied again.

@dolmen dolmen self-assigned this Sep 1, 2015
@dolmen
Copy link
Contributor Author

dolmen commented Sep 1, 2015

Unfortunately it seems at the moment I don't have full GitHub rights on this repo to reopen the issue despites I opened it and I'm the current CPAN maintainer.

@karenetheridge
Copy link
Member

Unfortunately it seems at the moment I don't have full GitHub rights on this repo to reopen the issue

doh, me neither. I think one of the admins of this github organization would need to go through each repository and update the list of collaborators to reflect the new membership of the group.

@dolmen
Copy link
Contributor Author

dolmen commented Sep 4, 2015

I now have full rights on the repo.
But it seems it is just not possible to reopen a pull request once the commits have been merged. I'll open another issue.

@dolmen dolmen mentioned this pull request Sep 4, 2015
Closed
dolmen added a commit that referenced this pull request Jan 29, 2016
@dolmen
podspell script: set output encoding from locale LC_CTYPE
e80e1f0 
Issue #22: reapply patch #17

Signed-off-by: Olivier Mengué <dolmen@cpan.org>
dolmen added a commit that referenced this pull request Jan 29, 2016
@dolmen
1.18
5616b4d 
	- New maintainer: Olivier Mengué (DOLMEN)
	  Repo is now hosted at github.com/perl-pod/Pod-Spell
	- Fix warning in strip_stopwords (GH #23)
	- re-apply "podspell script: set output encoding from locale CTYPE"
	  (GH #22, #17). The issue in 1.16 was an incorrect dependency
	  on I18N::Langinfo.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@dolmen dolmen

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@dolmen @karenetheridge @xenoterracide @dagolden