Overview
Examples
Screenshots
Comparisons
Applications
Download
Documentation
Tutorials
Bazaar
Status & Roadmap
FAQ
Authors & License
Forums
Funding Ultimate++
Search on this site
Search in forums












SourceForge.net Logo
Home » Developing U++ » UppHub » Added SVN Library
Added SVN Library [message #16435] Sun, 15 June 2008 02:57 Go to next message
mdelfede is currently offline  mdelfede
Messages: 1307
Registered: September 2007
Ultimate Contributor
Well, just a pre-release, many stuffs are missing but it's already usable.
I added a small application to test it, it just fetches a folder from upp svn three, allows to list it, change files, look at status, look at repository log, reconnect to folder, wipe it.

Can be used freely, it doesn't write back to upp repository (and, it CAN'T ! Smile ).
No docs for the moment, but source overcommented as usual Smile

Bazaar/SvnLib for the library
Bazaar/SvnTest for the test application.

Ciao

Max
Re: Added SVN Library [message #16476 is a reply to message #16435] Tue, 17 June 2008 13:23 Go to previous messageGo to next message
tojocky is currently offline  tojocky
Messages: 607
Registered: April 2008
Location: UK
Contributor

Error on build SVNTest (win32 XP with MSC8 optimal):

Quote:

----- SvnLib ( GUI GCC WIN32 ) (2 / 11)
SvnLib.cpp
In file included from D:\upp\svn\trunk\bazaar\SvnLib\SvnLib.cpp:3:
D:\upp\svn\trunk\bazaar\SvnLib\/SvnLib.h:125: error: 'ulong' does not name a type
D:\upp\svn\trunk\bazaar\SvnLib\/SvnLib.h:129: error: 'ulong' does not name a type
D:\upp\svn\trunk\bazaar\SvnLib\/SvnLib.h:194: error: 'ulong' does not name a type
D:\upp\svn\trunk\bazaar\SvnLib\/SvnLib.h:197: error: 'ulong' does not name a type
D:\upp\svn\trunk\bazaar\SvnLib\SvnLib.cpp: In member function 'void Upp::Svn::Init()':
D:\upp\svn\trunk\bazaar\SvnLib\SvnLib.cpp:195: error: 'FCheckedRevision' was not declared in this scope
D:\upp\svn\trunk\bazaar\SvnLib\SvnLib.cpp:198: error: 'FHeadRevision' was not declared in this scope
D:\upp\svn\trunk\bazaar\SvnLib\SvnLib.cpp: In member function 'Upp::Svn::Errors Upp::Svn::Connect(const Upp::String&)':
D:\upp\svn\trunk\bazaar\SvnLib\SvnLib.cpp:484: error: 'FCheckedRevision' was not declared in this scope
D:\upp\svn\trunk\bazaar\SvnLib\SvnLib.cpp:499: error: 'FHeadRevision' was not declared in this scope
D:\upp\svn\trunk\bazaar\SvnLib\SvnLib.cpp: In member function 'Upp::Svn::Errors Upp::Svn::Update(const Upp::String&)':
D:\upp\svn\trunk\bazaar\SvnLib\SvnLib.cpp:600: error: 'FCheckedRevision' was not declared in this scope
SvnLib: 1 file(s) built in (0:01.25), 1250 msecs / file, duration = 1266 msecs, parallelization 0%

There were errors. (1:32.29)
Re: Added SVN Library [message #16485 is a reply to message #16476] Tue, 17 June 2008 18:31 Go to previous messageGo to next message
mdelfede is currently offline  mdelfede
Messages: 1307
Registered: September 2007
Ultimate Contributor
uhmmm... I guess it's time I setup a machine with windows upp....

Max
Re: Added SVN Library [message #16486 is a reply to message #16435] Tue, 17 June 2008 18:41 Go to previous messageGo to next message
cocob is currently offline  cocob
Messages: 156
Registered: January 2008
Experienced Member
use virtualbox Smile
Re: Added SVN Library [message #16489 is a reply to message #16485] Tue, 17 June 2008 21:32 Go to previous messageGo to next message
mdelfede is currently offline  mdelfede
Messages: 1307
Registered: September 2007
Ultimate Contributor
Should be fixed... in MSC8 'ulong' is not defined, so changed with long.
Please test and give feedback !

Ciao

Max
Re: Added SVN Library [message #16490 is a reply to message #16489] Tue, 17 June 2008 22:13 Go to previous messageGo to next message
Oblivion is currently offline  Oblivion
Messages: 1091
Registered: August 2007
Senior Contributor
Great job!.

SvnLib compiles successfully on MinGW, but it fails to compile on MSVC9.0 with the following error:

Quote:


----- CtrlLib ( GUI MSC9 DEBUG DEBUG_FULL BLITZ WIN32 MSC ) (1 / 11)
----- SvnLib ( GUI MSC9 DEBUG DEBUG_FULL BLITZ WIN32 MSC ) (2 / 11)
SvnLib.cpp
c:\myapps\svnlib\svnlib.cpp(867) : error C4716: 'Upp::Svn::UpdateInfo' : must return a value
SvnLib: 1 file(s) built in (0:05.68), 5687 msecs / file, duration = 5703 msecs

There were errors. (0:05.84)



In SvnLib.cpp,

// updates info from current repository
Svn::Errors Svn::UpdateInfo(void)
{

} // END Svn::UpdateInfo()


Should return an error code. I'm suprised that the ever-complaining MinGW did not give any errors, not even a warning. Laughing


[Updated on: Tue, 17 June 2008 22:15]

Report message to a moderator

Re: Added SVN Library [message #16491 is a reply to message #16490] Wed, 18 June 2008 00:32 Go to previous messageGo to next message
mdelfede is currently offline  mdelfede
Messages: 1307
Registered: September 2007
Ultimate Contributor
Oblivion wrote on Tue, 17 June 2008 22:13

Great job!.

SvnLib compiles successfully on MinGW, but it fails to compile on MSVC9.0 with the following error:

Quote:


----- CtrlLib ( GUI MSC9 DEBUG DEBUG_FULL BLITZ WIN32 MSC ) (1 / 11)
----- SvnLib ( GUI MSC9 DEBUG DEBUG_FULL BLITZ WIN32 MSC ) (2 / 11)
SvnLib.cpp
c:\myapps\svnlib\svnlib.cpp(867) : error C4716: 'Upp::Svn::UpdateInfo' : must return a value
SvnLib: 1 file(s) built in (0:05.68), 5687 msecs / file, duration = 5703 msecs

There were errors. (0:05.84)



In SvnLib.cpp,

// updates info from current repository
Svn::Errors Svn::UpdateInfo(void)
{

} // END Svn::UpdateInfo()


Should return an error code. I'm suprised that the ever-complaining MinGW did not give any errors, not even a warning. Laughing


Well, well... It's not complete ! Smile
I'd like to know if demo app works on windows too, by now... Just tested on Linux Smile
I still must extend the class to manage tags/branches in some easy way, as other important stuffs.
By now the demo just allow to connect to upp (small part) of svn repo and do checkout/logs on it... If all is ok, I'll continue it.
BTW, for the error above, just add
return Svn::Ok;

to function, it's just a stub by now.

Thanx for testing !

Max

Re: Added SVN Library [message #16495 is a reply to message #16491] Wed, 18 June 2008 11:07 Go to previous messageGo to next message
Oblivion is currently offline  Oblivion
Messages: 1091
Registered: August 2007
Senior Contributor
There is another error on windows. The assertion fails in MSVC9.0. And on MinGW, test app gives "SVN::Unknown error".



This error occurs after any attempt to "connect" or "checkout". I've traced the process and marked the point where it gives the error, though I didn't have the time to examine the situation. But hopefully it should give you a clue.

In SysExec.cpp (line 235):

index.php?t=getfile&id=1239&private=0

I guess (only a guess) the error is probably due to a parsing issue.

Regards.



  • Attachment: error.jpg
    (Size: 114.93KB, Downloaded 624 times)


[Updated on: Wed, 18 June 2008 11:12]

Report message to a moderator

Re: Added SVN Library [message #16502 is a reply to message #16435] Wed, 18 June 2008 14:28 Go to previous messageGo to next message
mdelfede is currently offline  mdelfede
Messages: 1307
Registered: September 2007
Ultimate Contributor
Do you have command line svn.exe installed and inside path ?

Max
Re: Added SVN Library [message #16504 is a reply to message #16502] Wed, 18 June 2008 15:08 Go to previous messageGo to next message
Oblivion is currently offline  Oblivion
Messages: 1091
Registered: August 2007
Senior Contributor
Embarassed Oops, sorry I forgot to reinstall the command line svn. Now It's ok.

[Updated on: Wed, 18 June 2008 15:09]

Report message to a moderator

Re: Added SVN Library [message #16505 is a reply to message #16504] Wed, 18 June 2008 16:03 Go to previous messageGo to next message
tojocky is currently offline  tojocky
Messages: 607
Registered: April 2008
Location: UK
Contributor

Will be great to detect if command line is installed or not, and do not crashing application!
Re: Added SVN Library [message #16506 is a reply to message #16505] Wed, 18 June 2008 16:26 Go to previous messageGo to next message
mdelfede is currently offline  mdelfede
Messages: 1307
Registered: September 2007
Ultimate Contributor
tojocky wrote on Wed, 18 June 2008 16:03

Will be great to detect if command line is installed or not, and do not crashing application!


uhmmmm... which part of the phrase "Well, just a pre-release, many stuffs are missing but it's already usable" was not clear ? Smile

It's just a stuff for testing, not complete at all... I was interested mainly to see if we've got problems on windows version, as I couldn't test the 'spawnvpe' stuffs that are quite different from linux.

BTW, on next days I'll finish it.

Ciao

Max
Re: Added SVN Library [message #16507 is a reply to message #16504] Wed, 18 June 2008 16:27 Go to previous message
mdelfede is currently offline  mdelfede
Messages: 1307
Registered: September 2007
Ultimate Contributor
Oblivion wrote on Wed, 18 June 2008 15:08

Embarassed Oops, sorry I forgot to reinstall the command line svn. Now It's ok.


Perfect ! So now that the platform-dependent stuff appears to work, I can complete it Smile
Thanx very much for testing !

Max
Previous Topic: quicktabs vs tabbar
Next Topic: Shiny: C++ profiler - Ultimate++ package
Goto Forum:
  


Current Time: Thu Mar 28 16:42:00 CET 2024

Total time taken to generate the page: 0.01019 seconds