Bug #38507

Add prefix to code cache identifier

Added by Helmut Hummel 11 months ago. Updated 11 months ago.

Status:Resolved Start date:2012-06-30
Priority:Should have Due date:
Assignee:Helmut Hummel % Done:

100%

Category:-
Target version:6.0.0-alpha3
TYPO3 Version:6.0 Complexity:
PHP Version:5.3
Votes: 0

Description

Currently all identifier for php code caches are only a sha1 hash, so it is not possible to distinguish them in the filesystem.

Add some speaking prefix to the identifier.


Related issues

related to Core - Bug #38515: Build more PHP modules in travis environment Resolved 2012-07-01

Associated revisions

Revision 6dcbfbe2
Added by Helmut Hummel 11 months ago

[TASK] Add prefix to code cache identifier

Currently all identifier for php code caches are only a sha1 hash,
so it is not possible to distinguish them in the filesystem.

Add some speaking prefix to the identifier.

Change-Id: I12971d524489b2d31b07440867d6afbaef42d62c
Resolves: #38507
Releases: 6.0
Reviewed-on: http://review.typo3.org/12483
Reviewed-by: Christian Kuhn
Reviewed-by: Helmut Hummel
Tested-by: Helmut Hummel

Revision d0fb869d
Added by Helmut Hummel 11 months ago

[BUGFIX] Fix unit tests for changed identifier

Because of the identifiers being changed in the last
commit, the unit tests failed.

Adjust the tests according to the change.

Change-Id: I62a168ffc5fe63fffc205ff551a7f954c197bc5c
Resolves: #38507
Releases: 6.0
Reviewed-on: http://review.typo3.org/12484
Reviewed-by: Helmut Hummel
Tested-by: Helmut Hummel

Revision 4d2538de
Added by Helmut Hummel 11 months ago

[TASK] Build more PHP modules in travis environment

In order to execute certain unit tests special requirements
of the environment must be met. In particular several
additional PHP modules need to be installed.

Instead of hiding what is added to the environment,
use a helper script to not clutter the travis
configuration file, but at the same time make it
transparent there which PHP modules are built.

Change-Id: I7111cbd0f3d1d32d68df6eaf8c4b6176bfe82eff
Resolves: #38507
Releases: 6.0
Reviewed-on: http://review.typo3.org/12498
Reviewed-by: Helmut Hummel
Tested-by: Helmut Hummel

Revision 9304f846
Added by Helmut Hummel 10 months ago

[TASK] Add prefix to code cache identifier

Currently all identifier for php code caches are only a sha1 hash,
so it is not possible to distinguish them in the filesystem.

Add some speaking prefix to the identifier.

Change-Id: I12971d524489b2d31b07440867d6afbaef42d62c
Resolves: #38507
Releases: 6.0
Reviewed-on: http://review.typo3.org/12483
Reviewed-by: Christian Kuhn
Reviewed-by: Helmut Hummel
Tested-by: Helmut Hummel

Revision 4b207043
Added by Helmut Hummel 10 months ago

[BUGFIX] Fix unit tests for changed identifier

Because of the identifiers being changed in the last
commit, the unit tests failed.

Adjust the tests according to the change.

Change-Id: I62a168ffc5fe63fffc205ff551a7f954c197bc5c
Resolves: #38507
Releases: 6.0
Reviewed-on: http://review.typo3.org/12484
Reviewed-by: Helmut Hummel
Tested-by: Helmut Hummel

Revision cabf5d20
Added by Helmut Hummel 10 months ago

[TASK] Build more PHP modules in travis environment

In order to execute certain unit tests special requirements
of the environment must be met. In particular several
additional PHP modules need to be installed.

Instead of hiding what is added to the environment,
use a helper script to not clutter the travis
configuration file, but at the same time make it
transparent there which PHP modules are built.

Change-Id: I7111cbd0f3d1d32d68df6eaf8c4b6176bfe82eff
Resolves: #38507
Releases: 6.0
Reviewed-on: http://review.typo3.org/12498
Reviewed-by: Helmut Hummel
Tested-by: Helmut Hummel

History

Updated by Gerrit Code Review 11 months ago

  • Status changed from New to Under Review

Patch set 1 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/12483

Updated by Helmut Hummel 11 months ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100

Updated by Gerrit Code Review 11 months ago

  • Status changed from Resolved to Under Review

Patch set 1 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/12484

Updated by Helmut Hummel 11 months ago

  • Status changed from Under Review to Resolved

Updated by Gerrit Code Review 11 months ago

  • Status changed from Resolved to Under Review

Patch set 2 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/12498

Updated by Helmut Hummel 11 months ago

  • Status changed from Under Review to Resolved

Also available in: Atom PDF