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

Reorganize directory structure. #20

Merged
merged 1 commit into from Feb 24, 2015

Conversation

Projects
None yet
2 participants
@adherzog
Contributor

adherzog commented Feb 24, 2015

Move modules into lib/, script into bin/, and update Makefile.PL/MANIFEST with new paths.

(This had been submitted in #9, but it was closed before ever being applied.)

@karenetheridge

This comment has been minimized.

Member

karenetheridge commented Feb 24, 2015

This is good, except uri-test should stay where it is, as it is not intended to be installed for users. (If it were, it would have documentation, and be mentioned as an installable executable in EXE_FILES in Makefile.PL.)

@adherzog

This comment has been minimized.

Contributor

adherzog commented Feb 24, 2015

Thanks, @karenetheridge. I've moved uri-test back to its original location.

@karenetheridge

This comment has been minimized.

Member

karenetheridge commented Feb 24, 2015

Super! Could you possibly squash those commits into one and force-push your branch?

Reorganize directory structure.
Move modules into /lib and update Makefile.PL/MANIFEST with new paths.
@adherzog

This comment has been minimized.

Contributor

adherzog commented Feb 24, 2015

Oh! I should've thought of that. :) Squashed and pushed.

karenetheridge added a commit that referenced this pull request Feb 24, 2015

@karenetheridge karenetheridge merged commit 3851013 into libwww-perl:master Feb 24, 2015

@karenetheridge

This comment has been minimized.

Member

karenetheridge commented Feb 24, 2015

thank you very much!

@adherzog adherzog deleted the adherzog:reorganize-directory-structure branch Feb 24, 2015

karenetheridge added a commit that referenced this pull request Feb 24, 2015

@karenetheridge

This comment has been minimized.

Member

karenetheridge commented Feb 24, 2015

1.66 has been released. Thanks again!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment