search

PDLPorters / extutils-f77

ExtUtils::F77 Perl module
https://p3rl.org/ExtUtils::F77
Other
0 stars 0 forks source link

Tweaks #1

Closed mohawk2 closed 6 years ago

mohawk2 commented 6 years ago

This simplifies the one test, adds metadata pointing to this updated GH repo, and stops the module from breaking in a "place with a space".

karlglazebrook commented 6 years ago

I tested this, think we should merge it.

mohawk2 commented 6 years ago

@karlglazebrook Have you heard from @devel-chm lately? It's been a while.

karlglazebrook commented 6 years ago

No I have not heard from him either. Hope all is OK.

devel-chm commented 6 years ago

Hi All-

I've been very busy with no time for PDL work.

--Chris

On Sat, Oct 20, 2018 at 10:44 PM Karl Glazebrook notifications@github.com wrote:

No I have not heard from him either. Hope all is OK.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/PDLPorters/extutils-f77/pull/1#issuecomment-431640779, or mute the thread https://github.com/notifications/unsubscribe-auth/AAwNeUadqIaGf7ei5wXztWNJwqiaEMNAks5unAnJgaJpZM4TugYX .

karlglazebrook commented 6 years ago

No worries. Me too for the last few months.

If you have 5 secs to up my github privs on pdl/extutils-f77 that would be welcome.

Karl

On 22 Oct 2018, at 10:10 am, devel-chm notifications@github.com wrote:

Hi All-

I've been very busy with no time for PDL work.

--Chris

On Sat, Oct 20, 2018 at 10:44 PM Karl Glazebrook notifications@github.com wrote:

No I have not heard from him either. Hope all is OK.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/PDLPorters/extutils-f77/pull/1#issuecomment-431640779, or mute the thread https://github.com/notifications/unsubscribe-auth/AAwNeUadqIaGf7ei5wXztWNJwqiaEMNAks5unAnJgaJpZM4TugYX .

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/PDLPorters/extutils-f77/pull/1#issuecomment-431712471, or mute the thread https://github.com/notifications/unsubscribe-auth/AOnymlMiSoSYDdefGgUqz7snusX-QlVCks5unP7bgaJpZM4TugYX.

mohawk2 commented 6 years ago

@karlglazebrook I think you may already have rights to merge the PRs on anything in PDLPorters, including this. It looks like you also have the rights to release this (and other PDL-ish modules) onto CPAN. Do you feel like doing so?

karlglazebrook commented 6 years ago

It seems not, at least for PDLPorters/extutils-f77, see screenshot

can someone fix?

Karl

On 31 Oct 2018, at 1:27 pm, mohawk2 notifications@github.com wrote:

@karlglazebrook https://github.com/karlglazebrook I think you may already have rights to merge the PRs on anything in PDLPorters, including this. It looks like you also have the rights to release this (and other PDL-ish modules) onto CPAN. Do you feel like doing so?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/PDLPorters/extutils-f77/pull/1#issuecomment-434538934, or mute the thread https://github.com/notifications/unsubscribe-auth/AOnymsJ9_SFhYJCSts9K4bPYuN_QTDS8ks5uqQqigaJpZM4TugYX.

mohawk2 commented 6 years ago

@karlglazebrook Unfortunately the email->issue gateway drops attachments. Could you come to the web page and attach the screenshot? I've had a quick look and can't see anything that shows why you wouldn't be able to merge; you're showing here as a "Member" of PDLPorters.

mohawk2 commented 6 years ago

Breaking news: Karl was only a "member" of the organisation, but is now an "owner". You should be able to merge now? Are you also able to cut releases?

karlglazebrook commented 6 years ago

Merging this. All seems like good changes.

karlglazebrook commented 6 years ago

Yes! Thanks for doing that

I’ve merged your tweaks branch

Karl

On 5 Nov 2018, at 3:34 am, mohawk2 notifications@github.com wrote:

Breaking news: Karl was only a "member" of the organisation, but is now an "owner". You should be able to merge now? Are you also able to cut releases?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/PDLPorters/extutils-f77/pull/1#issuecomment-435684326, or mute the thread https://github.com/notifications/unsubscribe-auth/AOnymu_vd8Nucd5K7tAk0gnsHWWIcA-4ks5urxb4gaJpZM4TugYX.

mohawk2 commented 6 years ago

@karlglazebrook Great! Are you minded to release EU::F77 to CPAN? :-)

karlglazebrook commented 6 years ago

Not yet. I have someone threatening me with new patches which sound useful :)

Is there a cool way to go github->CPAN?

On 5 Nov 2018, at 10:46 am, mohawk2 notifications@github.com wrote:

@karlglazebrook https://github.com/karlglazebrook Great! Are you minded to release EU::F77 to CPAN? :-)

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/PDLPorters/extutils-f77/pull/1#issuecomment-435720124, or mute the thread https://github.com/notifications/unsubscribe-auth/AOnymnHTH-j7nBsaBMFqEbrzo_YdAsMTks5ur3xFgaJpZM4TugYX.

mohawk2 commented 6 years ago

You could release this one, and then take your time with the other patches. There's no cost per release :-)

To answer your question, there are various tools these days for easy uploading. However, I tend to stay old-school, and after running the various normal tests, I then do these steps manually:

karlglazebrook commented 6 years ago

It sounds like it would only be a day or two and a very minor patch

Yes I can do all that, I was hoping there would be a button on CPAN or GITHUB to release to CPAN :)

On 5 Nov 2018, at 12:31 pm, mohawk2 notifications@github.com wrote:

You could release this one, and then take your time with the other patches. There's no cost per release :-)

To answer your question, there are various tools these days for easy uploading. However, I tend to stay old-school, and after running the various normal tests, I then do these steps manually:

increment the version update Changes as appropriate re-run perl Makefile.PL git commit -am v1.21 git tag v1.21 make disttest make dist cpan-upload ExtUtils- rm ExtUtils- git push git push --tags — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/PDLPorters/extutils-f77/pull/1#issuecomment-435729644, or mute the thread https://github.com/notifications/unsubscribe-auth/AOnymtMU7zmKi6zN41HXgxc7RXKKDvdlks5ur5UAgaJpZM4TugYX.

mohawk2 commented 6 years ago

I didn't doubt you! I was just noting it down for myself. I should really automate all of that.

Sadly, no cool button for now. It would probably be possible to make Travis take every successful build of master and release it as a new version on CPAN, but not quite trivially.

karlglazebrook commented 6 years ago

Travis?

On 5 Nov 2018, at 12:39 pm, mohawk2 notifications@github.com wrote:

I didn't doubt you! I was just noting it down for myself. I should really automate all of that.

Sadly, no cool button for now. It would probably be possible to make Travis take every successful build of master and release it as a new version on CPAN, but not quite trivially.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/PDLPorters/extutils-f77/pull/1#issuecomment-435730533, or mute the thread https://github.com/notifications/unsubscribe-auth/AOnymh_1BvTag4L2TjzY68VNb4F_9tw6ks5ur5bWgaJpZM4TugYX.

mohawk2 commented 6 years ago

Sorry - continuous integration service. It's not even been set up for this repo. One possibility with continuous integration is to do follow-on steps on success. With enough tests, one can reach a level of software infrastructure known as "continuous delivery" where a merge leads to a newly-released version. We aren't near there yet with this repo! Quite a few things would need adjusting.

karlglazebrook commented 6 years ago

Let’s not go there yet!

Karl

On 5 Nov 2018, at 1:55 pm, mohawk2 notifications@github.com wrote:

Sorry - continuous integration service. It's not even been set up for this repo. One possibility with continuous integration is to do follow-on steps on success. With enough tests, one can reach a level of software infrastructure known as "continuous delivery" where a merge leads to a newly-released version. We aren't near there yet with this repo! Quite a few things would need adjusting.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/PDLPorters/extutils-f77/pull/1#issuecomment-435740627, or mute the thread https://github.com/notifications/unsubscribe-auth/AOnymkitzgKuyci2hKy5zC0mg2yR3oBkks5ur6iagaJpZM4TugYX.