? ? Success

User tests: Successful: Unsuccessful:

avatar andrepereiradasilva
andrepereiradasilva
8 Apr 2016

Pull Request for Improvement.

Summary of Changes

This PR adds a new CLI script to make the version bumps auto.

Linux bash usage examples:

# Inside joomla root folder.
php build/bump.php -v 3.5.5-dev
php build/bump.php -v 3.5.5-beta1
php build/bump.php -v 3.5.5-beta1-dev
php build/bump.php -v 3.5.5-beta2
php build/bump.php -v 3.5.5-rc1
php build/bump.php -v 3.5.5
php build/bump.php -v 3.6.0 -c Unicorn
php build/bump.php -v 3.6.0 -c "My Custom Codename"

# Inside joomla build folder.
php bump.php -v 3.5.5-dev

# Anywhere
/usr/bin/php /path/to/joomla-cms/build/bump.php -v 3.5.5
Result example

bump

Testing Instructions

  1. Apply patch with the patchtester.
  2. Run CLI script with a version of your choice.
  3. Check if the version variables/strings are changed in all files. ➤ Files that should change

Observations

If anything is missing or incorrect please say.

As it is, this does not tag the version bump, neither uploads to github. Just changes the files.

Suggestions and/or improvements are welcome.

avatar andrepereiradasilva andrepereiradasilva - open - 8 Apr 2016
avatar andrepereiradasilva andrepereiradasilva - change - 8 Apr 2016
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 8 Apr 2016
Labels Added: ?
avatar andrepereiradasilva andrepereiradasilva - change - 8 Apr 2016
The description was changed
avatar brianteeman brianteeman - change - 8 Apr 2016
Category CLI
avatar wilsonge
wilsonge - comment - 8 Apr 2016

I will test this this evening :) Looks nice tho :)

avatar andrepereiradasilva andrepereiradasilva - change - 8 Apr 2016
The description was changed
avatar andrepereiradasilva
andrepereiradasilva - comment - 8 Apr 2016

ok, thanks. Just have one question the BUILD in version.php is used for what?

avatar wilsonge
wilsonge - comment - 8 Apr 2016

I don't think it is being used to the best of my knowledge. @mbabker ?

avatar andrepereiradasilva andrepereiradasilva - change - 8 Apr 2016
The description was changed
avatar photodude
photodude - comment - 8 Apr 2016

If we don't already have it, it would be nice to see something like this for the copyright dates too.
The more we can automate the better,

avatar 810 810 - test_item - 8 Apr 2016 - Tested successfully
avatar 810
810 - comment - 8 Apr 2016

I have tested this item :white_check_mark: successfully on 0e811db

Tested with Windows Powershell + windows Bash


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/9804.

avatar brianteeman
brianteeman - comment - 8 Apr 2016

If we don't already have it, it would be nice to see something like this for the copyright dates too.

@infograf768 usually does that


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/9804.

avatar mbabker
mbabker - comment - 8 Apr 2016

I still want to know why that copyright statement has to be in every file
(years inclusive) and can't be moved into a LICENSE or COPYRIGHT file.
Especially because there's no way to claim copyright in 2005 of a file
created in 2015.

On Friday, April 8, 2016, Walt Sorensen notifications@github.com wrote:

If we don't already have it, it would be nice to see something like this
for the copyright dates too.
The more we can automate the better,


You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
#9804 (comment)

avatar wilsonge
wilsonge - comment - 8 Apr 2016
Referencing GNU's guide on it:

https://www.gnu.org/prep/maintain/html_node/Copyright-Notices.html

"You should maintain a proper copyright notice and a license notice in each nontrivial file in the package. (Any file more than ten lines long is nontrivial for this purpose.) This includes header files and interface definitions for building or running the program, documentation files, and any supporting files. If a file has been explicitly placed in the public domain, then instead of a copyright notice, it should have a notice saying explicitly that it is in the public domain."

So the quick answer is - yes.

Referencing further background:

http://fairuse.stanford.edu/overview/faqs/copyright-protection/

https://www.law.cornell.edu/wex/copyright

https://www.copyrightservice.co.uk/copyright/p01_uk_copyright_law

The reasoning is that with current copyright laws the copyright holds for X years.

The X is different from country to country but in most cases Copyright based on publication and creation dates (as ours is) is 50-70 years.

There is an extensive list here:

https://en.wikipedia.org/wiki/List_of_countries%27_copyright_lengths

So now the question remains - Joomla was originally released in 2005 - does anyone care about Joomla copyright in 2055 or 2075 depending the country?

I will _assume_ that WP, Drupal et. al. are not stipulating the year because they dont think the copyright is relevant 50 or 70 years into the future.

But if we follow the GNU guide it should be there.

So i hope that gives some background and insights - as for IF we loose copyright by not stating it - i dont _think_ so, considering Google etc. and many others does not do it, it just prevents the copyright duration to be a runningly updated 70 years (as most common for software).
avatar richard67
richard67 - comment - 8 Apr 2016

Hmm, I am not sure if Jean Marie is willing to update the copyrights manually for the next 70 years :tongue:


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/9804.

avatar richard67
richard67 - comment - 8 Apr 2016

Year by year I meant :smile:


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/9804.

avatar mbabker
mbabker - comment - 9 Apr 2016

So it's not good enough to say "copyright OSM, see COPYRIGHT.txt for
additional details" in the file headers like we basically do with the
license? Every file has to have a copyright claim and said claim uses the
project's copyright dates versus the date (year) the code would first be
copyrighted to OSM? That's what irks me, I totally get having the
copyright claims but I still don't get the WHY as to it being required in
the current format.

On Friday, April 8, 2016, George Wilson notifications@github.com wrote:

Referencing GNU's guide on it:
https://www.gnu.org/prep/maintain/html_node/Copyright-Notices.html

"You should maintain a proper copyright notice and a license notice in each nontrivial file in the package. (Any file more than ten lines long is nontrivial for this purpose.) This includes header files and interface definitions for building or running the program, documentation files, and any supporting files. If a file has been explicitly placed in the public domain, then instead of a copyright notice, it should have a notice saying explicitly that it is in the public domain."

So the quick answer is - yes.

Referencing further background:
http://fairuse.stanford.edu/overview/faqs/copyright-protection/
https://www.law.cornell.edu/wex/copyright
https://www.copyrightservice.co.uk/copyright/p01_uk_copyright_law

The reasoning is that with current copyright laws the copyright holds for X years.

The X is different from country to country but in most cases Copyright based on publication and creation dates (as ours is) is 50-70 years.

There is an extensive list here:
https://en.wikipedia.org/wiki/List_of_countries%27_copyright_lengths

So now the question remains - Joomla was originally released in 2005 - does anyone care about Joomla copyright in 2055 or 2075 depending the country?

I will assume that WP, Drupal et. al. are not stipulating the year because they dont think the copyright is relevant 50 or 70 years into the future.

But if we follow the GNU guide it should be there.

So i hope that gives some background and insights - as for IF we loose copyright by not stating it - i dont think so, considering Google etc. and many others does not do it, it just prevents the copyright duration to be a runningly updated 70 years (as most common for software).


You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
#9804 (comment)

avatar richard67
richard67 - comment - 9 Apr 2016

@andrepereiradasilva Sorry I cannot help with testing. Still use my remote shared host for testing where I have my website on, and there I cannot run CLI because of restrictions from service provider.

avatar andrepereiradasilva
andrepereiradasilva - comment - 9 Apr 2016

no problem, i think @wilsonge will also test

if you want to test other PR :wink:, i have some needing tests https://github.com/joomla/joomla-cms/pulls/andrepereiradasilva

avatar brianteeman brianteeman - test_item - 14 Apr 2016 - Tested unsuccessfully
avatar brianteeman
brianteeman - comment - 14 Apr 2016

I have tested this item :red_circle: unsuccessfully on 0e811db

My syntax was

php bump.php -v 4.0.1 - c "Joomla 4 the next wonder of the earth"

The following errors were found when checking the files
/administrator/manifests/packages/pkg_en-GB.xml

<version>4.0.1.1</version>

/libraries/cms/version/version.php

const CODENAME = 'Unicorn';


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/9804.

avatar andrepereiradasilva
andrepereiradasilva - comment - 14 Apr 2016

The following errors were found when checking the files
/administrator/manifests/packages/pkg_en-GB.xml
4.0.1.1

That was an @infograf768 request.
As i told before there are inconsistencies in the en-GB language pack. The one in PR #9835 and the fact that the en-GB language extensions manifests have 3 version digits when all other language packs have 4 version digits.
IMO, for consistency, en-GB language should behave exactly like the other language packs, with the simple difference that cannot be uninstalled, i.e., the 3 extensions (2 language + 1 package) are blocked. But that is other decision.

/libraries/cms/version/version.php
const CODENAME = 'Unicorn';

You cli command is not correct.
It should be (note the non existent space in the "-c" option):

php bump.php -v 4.0.1 -c "Joomla 4 the next wonder of the earth"
avatar brianteeman brianteeman - test_item - 14 Apr 2016 - Tested successfully
avatar brianteeman
brianteeman - comment - 14 Apr 2016

I have tested this item :white_check_mark: successfully on 0e811db

Now I remember the language issue and good spot on my typo

All good


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/9804.

avatar brianteeman brianteeman - change - 14 Apr 2016
Status Pending Ready to Commit
avatar brianteeman
brianteeman - comment - 14 Apr 2016

RTC

I know its a NEW feature so should be 3.6 but as this is an internal tool only and not distributed I am setting it as 3.5.2


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/9804.

avatar joomla-cms-bot joomla-cms-bot - change - 14 Apr 2016
Labels Added: ?
avatar infograf768
infograf768 - comment - 14 Apr 2016

@brianteeman
The TTs language xmls version can be either 3.x.x or 3.x.x.x (where the last x shows the version of the pack installed or available). This is used only for display. en-GB does not need the 4th digit for the display.
BUT the pkg.xml for the TTs has to be with 4 digits to be used in the cron job to propose updates.
As this "may" be a model for users, I indeed suggested to keep there the same format as the TT's one.

avatar brianteeman brianteeman - change - 14 Apr 2016
Milestone Added:
avatar wilsonge wilsonge - close - 14 Apr 2016
avatar wilsonge wilsonge - merge - 14 Apr 2016
avatar joomla-cms-bot joomla-cms-bot - close - 14 Apr 2016
avatar wilsonge wilsonge - reference | ee6dd6f - 14 Apr 16
avatar wilsonge wilsonge - merge - 14 Apr 2016
avatar wilsonge wilsonge - close - 14 Apr 2016
avatar brianteeman brianteeman - close - 14 Apr 2016
avatar wilsonge wilsonge - change - 14 Apr 2016
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2016-04-14 12:30:27
Closed_By wilsonge
avatar joomla-cms-bot joomla-cms-bot - change - 14 Apr 2016
Labels Removed: ?
avatar andrepereiradasilva andrepereiradasilva - head_ref_deleted - 14 Apr 2016
avatar rdeutz rdeutz - change - 1 May 2016
Milestone Removed:
avatar rdeutz rdeutz - change - 1 May 2016
Milestone Added:
avatar rdeutz rdeutz - change - 1 May 2016
Milestone Added:
avatar rdeutz rdeutz - change - 1 May 2016
Milestone Removed:
avatar brianteeman brianteeman - change - 9 May 2016
Labels Added: ?

Add a Comment

Login with GitHub to post a comment